Bug 12344: QA followup: use next instead of return

Use next instead of return when generating templates.
In case patron has enabled a message type that misses a template,
next message type will be attempted instead of returning at once.

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

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 1cc7c5b..02c799a 100644 (file)
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -3274,7 +3274,7 @@ sub SendCirculationAlert {
                     'borrowers'   => $borrower,
                     'branches'    => $branch,
                 }
-            ) or return;
+            ) or next;
             C4::Message->enqueue($letter, $borrower, $_);
         } else {
             #warn "append to old message";
@@ -3291,7 +3291,7 @@ sub SendCirculationAlert {
                     'borrowers'   => $borrower,
                     'branches'    => $branch,
                 }
-            ) or return;
+            ) or next;
             $message->append($letter);
             $message->update;
         }