From e4d88bcf43996f1d118427067fe2e06a272167d9 Mon Sep 17 00:00:00 2001 From: Benjamin Rokseth 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 Signed-off-by: Tomas Cohen Arazi --- 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