From 4553f9c59ae076aa447ef9aa5df0e4861953d92e Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Thu, 24 Apr 2025 13:45:32 +0300 Subject: [PATCH] Bug 34776: Remove unneeded if statement We don't need to check if $op is add_form after error, since this value always changes. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind --- members/memberentry.pl | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/members/memberentry.pl b/members/memberentry.pl index 348c90019e..59bffba29e 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -845,11 +845,7 @@ if ( C4::Context->preference('EnhancedMessagingPreferences') ) { C4::Form::MessagingPreferences::set_form_values( { borrowernumber => $borrowernumber }, $template ); } } else { - if ( $op eq 'add_form' ) { - C4::Form::MessagingPreferences::restore_form_values( $input, $template ); - } else { - C4::Form::MessagingPreferences::restore_form_values( $input, $template ); - } + C4::Form::MessagingPreferences::restore_form_values( $input, $template ); } $template->param( SMSSendDriver => C4::Context->preference("SMSSendDriver") ); $template->param( SMSnumber => $data{'smsalertnumber'} ); -- 2.39.5