[koha-commits] main Koha release repository branch master updated. v3.12.00-beta1-829-gff7f372

[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, master has been updated
       via  ff7f37202949bb6d60cdf30de4e237e6b1de93eb (commit)
      from  057297e2bf0686e8272bff6d2b488fe928a7a2ad (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 ff7f37202949bb6d60cdf30de4e237e6b1de93eb
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Tue Jul 9 17:38:04 2013 +0200

    Bug 9362: fix query that finds upcoming due loans
    
    C4:Circulation:GetUpcomingDueIssues is used in the advance_notices.pl
    script.  This patch corrects an error in its handling of the maxdays
    parameter that resulted in it picking up *all* upcoming due loans and
    recently overdue loans.
    
    Test plan :
    - Create an issue with a date due in the paste
    - Create an issue with a date due in two days
    - Launch advance notices with due date in max 2 days : perl misc/cronjobs/advance_notices.pl -c -n -v -m=2
    => You get a warn "found 0 issues"
    - Launch advance notices with due date in max 3 days : perl misc/cronjobs/advance_notices.pl -c -n -v -m=3
    => You get a warn "found 1 issues"
    
    Signed-off-by: Mathieu Saby <mathieu.saby at univ-rennes2.fr>
    I did the following test :
    
    - 1 book to check in 2 days
    - 2 books to check in in the past
    
    before applying the patch :
    
    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=2
    getting upcoming due issues at ../misc/cronjobs/advance_notices.pl line 203.
    found 1 issues at ../misc/cronjobs/advance_notices.pl line 205.
    
    I changed the value of "-m" : 0, 1, 2, 3, 4
    => always 1 issue found (the book to check in in 2 days)
    
    after applying the patch :
    
    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=2
    found 0 issues
    for m = 0, 1, 2 => 0 issues
    
    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=3
    found 1 issues
    for m = 3,4,5 => 1 issues (the book to check in in 2 days)
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Passes koha-qa.pl, works as advertised.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 C4/Circulation.pm |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
main Koha release repository