Bugzilla – Attachment 48065 Details for
Bug 9021
Add SMS via email as an alternative to SMS services via SMS::Send drivers
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 9021: Save the provider even if the sms number is not modified
Bug-9021-Save-the-provider-even-if-the-sms-number-.patch (text/plain), 1.38 KB, created by
Jonathan Druart
on 2016-02-15 16:30:27 UTC
(
hide
)
Description:
Bug 9021: Save the provider even if the sms number is not modified
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2016-02-15 16:30:27 UTC
Size:
1.38 KB
patch
obsolete
>From 9753caeeb266c64f961f689387a55956cd1835ee Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 15 Feb 2016 16:26:53 +0000 >Subject: [PATCH] Bug 9021: Save the provider even if the sms number is not > modified > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >--- > opac/opac-messaging.pl | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > >diff --git a/opac/opac-messaging.pl b/opac/opac-messaging.pl >index 3ea529e..1a07f13 100755 >--- a/opac/opac-messaging.pl >+++ b/opac/opac-messaging.pl >@@ -50,11 +50,13 @@ my $messaging_options = C4::Members::Messaging::GetMessagingOptions(); > > if ( defined $query->param('modify') && $query->param('modify') eq 'yes' ) { > my $sms = $query->param('SMSnumber'); >- if ( defined $sms && ( $borrower->{'smsalertnumber'} // '' ) ne $sms ) { >+ my $sms_provider_id = $query->param('sms_provider_id'); >+ if ( defined $sms && ( $borrower->{'smsalertnumber'} // '' ) ne $sms >+ or ( $borrower->{sms_provider_id} // '' ) ne $sms_provider_id ) { > ModMember( > borrowernumber => $borrowernumber, > smsalertnumber => $sms, >- sms_provider_id => $query->param('sms_provider_id'), >+ sms_provider_id => $sms_provider_id, > ); > $borrower = C4::Members::GetMember( borrowernumber => $borrowernumber ); > } >-- >2.7.0
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 9021
:
13290
|
13291
|
13293
|
13294
|
14166
|
14167
|
14363
|
15685
|
15687
|
15689
|
15691
|
16628
|
17311
|
21729
|
22478
|
28488
|
29927
|
32453
|
34180
|
34181
|
35816
|
40720
|
44331
|
45152
|
46163
|
46164
|
46165
|
46168
|
46169
|
46170
|
46171
|
47128
|
47129
|
47130
|
47133
|
47134
|
47135
|
47338
|
47339
|
47814
|
47815
|
47816
|
47817
|
47818
|
47845
|
47846
|
47847
|
47848
|
47995
|
48024
|
48025
|
48026
|
48027
|
48028
|
48029
|
48030
|
48031
|
48032
|
48033
|
48034
|
48035
|
48036
|
48052
|
48053
|
48054
|
48055
|
48056
|
48057
|
48058
|
48059
|
48060
|
48061
|
48062
|
48063
|
48064
|
48065
|
48066