Bugzilla – Attachment 57669 Details for
Bug 17419
Fix more confusion between smsalertnumber and mobile
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 17419: Fix smsalertnumber and mobile confusion in moremember.pl
PASSED-QA-Bug-17419-Fix-smsalertnumber-and-mobile-.patch (text/plain), 2.24 KB, created by
Katrin Fischer
on 2016-11-20 19:27:28 UTC
(
hide
)
Description:
[PASSED QA] Bug 17419: Fix smsalertnumber and mobile confusion in moremember.pl
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2016-11-20 19:27:28 UTC
Size:
2.24 KB
patch
obsolete
>From 02de90b02f753e607a59a2c622cb79aa6389e8a9 Mon Sep 17 00:00:00 2001 >From: Lari Taskula <lari.taskula@jns.fi> >Date: Mon, 10 Oct 2016 17:06:07 +0300 >Subject: [PATCH] [PASSED QA] Bug 17419: Fix smsalertnumber and mobile > confusion in moremember.pl > >members/moremember.pl will set mobile number as smsalertnumber in template if >smsalertnumber is not defined. This will cause incorrect display for SMS number >in patron's Details-tab. This confusion between smsalertnumber and mobile is >already fixed in Bug 14683, but members/moremember.pl was not fixed yet. > >This is a minor issue since it won't occur for manually added new patrons due to >fixes already pushed in Bug 14683, but in case patron's smsalertnumber is null >in database, this bug can be replicated: > >To test: >1. Set EnhancedMessagingPreferences to "Allow" and make sure SMSSendDriver > has been set. >2. Add a new patron, give it a mobile/other phone number. >3. Run a SQL query: > update koha.borrowers set smsalertnumber=NULL where borrowernumber=XXX; > (replace XXX with your new patron's borrowernumber) >4. Go to patron's details tab and observe that SMS number shows the mobile/ > other phone you provided earlier. >5. Apply patch. >6. Refresh patron's details tab. >7. Observe that smsalertnumber is now empty, as it should be. > >Followed test plan, works as expected. >Signed-off-by: Marc <veron@veron.ch> > >Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> >--- > members/moremember.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/members/moremember.pl b/members/moremember.pl >index 01d16b6..0661d0f 100755 >--- a/members/moremember.pl >+++ b/members/moremember.pl >@@ -318,7 +318,7 @@ if (C4::Context->preference('EnhancedMessagingPreferences')) { > C4::Form::MessagingPreferences::set_form_values({ borrowernumber => $borrowernumber }, $template); > $template->param(messaging_form_inactive => 1); > $template->param(SMSSendDriver => C4::Context->preference("SMSSendDriver")); >- $template->param(SMSnumber => defined $data->{'smsalertnumber'} ? $data->{'smsalertnumber'} : $data->{'mobile'}); >+ $template->param(SMSnumber => $data->{'smsalertnumber'}); > $template->param(TalkingTechItivaPhone => C4::Context->preference("TalkingTechItivaPhoneNotification")); > } > >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17419
:
56106
|
56120
| 57669