From 23671192ffbc8db1eb326967f4bc4484bde783ca Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Wed, 18 Dec 2019 18:53:04 +0000
Subject: [PATCH] Bug 24268: Break assignment and warn into separate actions

---
 misc/cronjobs/advance_notices.pl | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/misc/cronjobs/advance_notices.pl b/misc/cronjobs/advance_notices.pl
index 7c7746cdbf..d8dcf73a51 100755
--- a/misc/cronjobs/advance_notices.pl
+++ b/misc/cronjobs/advance_notices.pl
@@ -582,7 +582,11 @@ sub send_digests {
                     branchcode     => $branchcode,
                     message_transport_type => $transport
                 }
-            ) || warn "no letter of type '$params->{letter_type}' found for borrowernumber $borrowernumber. Please see sample_notices.sql";
+            );
+            unless ( $letter ){
+                warn "no letter of type '$params->{letter_type}' found for borrowernumber $borrowernumber. Please see sample_notices.sql";
+                next;
+            }
             push @letters, $letter if $letter;
         }
 
-- 
2.11.0