From e4d88bcf43996f1d118427067fe2e06a272167d9 Mon Sep 17 00:00:00 2001
From: Benjamin Rokseth <benjamin.rokseth@kul.oslo.kommune.no>
Date: Thu, 5 Mar 2015 12:18:09 +0100
Subject: [PATCH] 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>
---
 C4/Circulation.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 1cc7c5b5e9..02c799ad40 100644
--- 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;
         }
-- 
2.39.5