@@ -, +, @@ --- opac/opac-messaging.pl | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) --- a/opac/opac-messaging.pl +++ a/opac/opac-messaging.pl @@ -48,10 +48,9 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user( my $borrower = GetMemberDetails( $borrowernumber ); my $messaging_options = C4::Members::Messaging::GetMessagingOptions(); -if ( defined $query->param('modify') && $query->param('modify') eq 'yes' ) { - - # If they've modified the SMS number, record it. - if ( ( defined $query->param('SMSnumber') ) && ( $query->param('SMSnumber') ne $borrower->{'mobile'} ) ) { +if ( defined $query->param('modify') && $query->param('modify') eq 'yes' ) { + + if ( $query->param('SMSnumber') ne $borrower->{'mobile'} ) { ModMember( borrowernumber => $borrowernumber, smsalertnumber => $query->param('SMSnumber') ); $borrower = GetMemberDetails( $borrowernumber ); @@ -65,7 +64,7 @@ C4::Form::MessagingPreferences::set_form_values({ borrowernumber => $borrowe # warn( Data::Dumper->Dump( [ $messaging_options ], [ 'messaging_options' ] ) ); $template->param( BORROWER_INFO => [ $borrower ], messagingview => 1, - SMSnumber => defined $borrower->{'smsalertnumber'} ? $borrower->{'smsalertnumber'} : $borrower->{'mobile'}, + SMSnumber => $borrower->{'smsalertnumber'} ? $borrower->{'smsalertnumber'} : $borrower->{'mobile'}, SMSSendDriver => C4::Context->preference("SMSSendDriver"), TalkingTechItivaPhone => C4::Context->preference("TalkingTechItivaPhoneNotification") ); --