Attachment 42050 Details for Bug 14683 – Bug 14683: Unable to clear SMS number

[patch] Bug 14683: Unable to clear SMS number

Bug-14683-Unable-to-clear-SMS-number.patch (text/plain), 2.17 KB, created by Marcel de Rooy on 2015-08-28 08:17:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2015-08-28 08:17:45 UTC

Size: 2.17 KB

patch

obsolete

>From 17dbb64219737c7a095ec5306f78f08a8dd179b5 Mon Sep 17 00:00:00 2001
>From: =?utf-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= <j.kylmala@gmail.com>
>Date: Wed, 19 Aug 2015 11:42:50 +0000
>Subject: [PATCH] Bug 14683: Unable to clear SMS number
>Content-Type: text/plain; charset=utf-8
>
>Enables to clear SMS number.
>
>To test:
>1. Go to opac-messaging.pl
>2. Insert SMS number and submit
>3. Clear SMS number and submit
>4. Observe that the sms number did not change
>5. Apply patch
>6. Clear SMS number and submit
>7. Observe that the sms number changes
>
>Sponsored-by: Vaara-kirjastot
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Adding a follow-up.
>---
> opac/opac-messaging.pl |    5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
>diff --git a/opac/opac-messaging.pl b/opac/opac-messaging.pl
>index 69ae517..b3b192c 100755
>--- a/opac/opac-messaging.pl
>+++ b/opac/opac-messaging.pl
>@@ -50,8 +50,7 @@ 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 ( $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") );
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 14683: 41664 | 42050 | 42051 | 42162 | 42187 | 42188 | 42189