From 0975507afdf0177083aaf1126518d97b29425f96 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Thu, 31 Oct 2024 16:55:21 +0000 Subject: [PATCH] Bug 30300: (QA follow-up) Remove extra return The routine loops on the found transports, if there are none it will simply return, no need to short circuit Signed-off-by: Nick Clemens --- Koha/Patron.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/Koha/Patron.pm b/Koha/Patron.pm index 1cd2534c8a2..6a67fa13f4c 100644 --- a/Koha/Patron.pm +++ b/Koha/Patron.pm @@ -2641,7 +2641,6 @@ sub queue_notice { } ); @message_transports = ( keys %{ $messaging_prefs->{transports} } ); $letter_code = $messaging_prefs->{transports}->{ $message_transports[0] } unless $letter_code; - return if ( $params->{message_name} eq 'Patron_Expiry' && !$messaging_prefs ); } else { @message_transports = @{$params->{message_transports}}; } -- 2.39.5