Bug 9016: (follow-up) treat missing transports for overdue notices as warnings
authorGalen Charlton <gmc@esilibrary.com>
Fri, 2 May 2014 21:11:30 +0000 (21:11 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Fri, 2 May 2014 21:11:30 +0000 (21:11 +0000)
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
misc/cronjobs/overdue_notices.pl

index 855558d..2a9f8ae 100755 (executable)
@@ -593,10 +593,8 @@ END_SQL
                     );
                     unless ($letter) {
                         $verbose and warn "Message '$overdue_rules->{letter$i}' content not found";
-
-                        # might as well skip while PERIOD, no other borrowers are going to work.
-                        # FIXME : Does this mean a letter must be defined in order to trigger a debar ?
-                        next PERIOD;
+                        # this transport doesn't have a configured notice, so try another
+                        next;
                     }
 
                     if ( $exceededPrintNoticesMaxLines ) {