[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.06-4-g24a6ef2

[Git repo owner]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 17.05.x has been updated
       via  24a6ef2ba16f18923a5fd9d9439cfe7c84d9ca20 (commit)
       via  dd2f28fab4853d917cc1c47e8fab5351335171fa (commit)
      from  efb28c4af16efbe688322415ae06c6a85fd3454b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 24a6ef2ba16f18923a5fd9d9439cfe7c84d9ca20
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Mon Aug 28 12:13:26 2017 +0200

    Bug 19190: [Follow-up] Do not rely on the return of ModItem
    
    touch_all_items looks at the return of ModItem to determine if the
    operation was successful. But ModItem does not return a meaningful
    value. This patch puts the ModItem call in an eval and looks at $@.
    
    Test plan:
    Run touch_all_items with a where condition and verbose option.
    Put print 1/0; at the end of ModItem.
    Run touch_all_items again. You should see: ERROR WITH ITEM xxx !!!!
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: David Bourgault <david.bourgault at inlibro.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 1fc0b652548510ef00a7e072cdf923357d5610aa)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit dd2f28fab4853d917cc1c47e8fab5351335171fa
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Mon Aug 28 11:59:19 2017 +0200

    Bug 19190: Silly calculation of average time in touch_all scripts
    
    When you want to calculate average time, do not divide count by time :)
    
    Test plan:
    Run the script with a where condition and verbose option and see that
    the average time is meaningful.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: David Bourgault <david.bourgault at inlibro.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit b3b86caaa5473ea51dc7223146e4c7cf0db431dc)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

-----------------------------------------------------------------------

Summary of changes:
 misc/maintenance/touch_all_biblios.pl |    2 +-
 misc/maintenance/touch_all_items.pl   |    5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
main Koha release repository