Bug 13679 : Bug in listing overdues
authorChris Cormack <chris@bigballofwax.co.nz>
Sat, 7 Feb 2015 04:04:40 +0000 (17:04 +1300)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Fri, 20 Feb 2015 13:19:36 +0000 (10:19 -0300)
To test
1/ Create some overdues and some issues due in the future
2/ Run the overdues script
3/ Notice item due in the future is in the list
4/ Apply patch
5/ Run script again
6/ Notice in the future not in the list

Signed-off-by: Nick <Nick@quechelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>

misc/cronjobs/overdue_notices.pl

index 4d05ff7..8db858d 100755 (executable)
@@ -602,6 +602,8 @@ END_SQL
                 my $j = 0;
                 my $exceededPrintNoticesMaxLines = 0;
                 while ( my $item_info = $sth2->fetchrow_hashref() ) {
+                    next unless ( DateTime->compare( $date_to_run,  dt_from_string($item_info->{date_due})) ) == 1;
+
                     if ( C4::Context->preference('OverdueNoticeCalendar') ) {
                         my $calendar =
                           Koha::Calendar->new( branchcode => $branchcode );