Bugzilla – Attachment 139938 Details for
Bug 31481
Pass message_id to SMS drivers
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 31481: pass message_id to SMS drivers
0001-Bug-31481-pass-message_id-to-SMS-drivers.patch (text/plain), 1.50 KB, created by
Johanna Räisä
on 2022-08-29 07:39:28 UTC
(
hide
)
Description:
Bug 31481: pass message_id to SMS drivers
Filename:
MIME Type:
Creator:
Johanna Räisä
Created:
2022-08-29 07:39:28 UTC
Size:
1.50 KB
patch
obsolete
>From 94c3eca18242a5db65ba72f9c351cd0b83f5ea9e Mon Sep 17 00:00:00 2001 >From: Johanna Raisa <johanna.raisa@gmail.com> >Date: Fri, 20 May 2022 10:16:10 +0300 >Subject: [PATCH] Bug 31481: pass message_id to SMS drivers > >This patch adds message_id to _send_message_by_sms and sms_send >so the driver could be build to catch SMS gateways delivery report. > >Test plan: >1) Apply the patch >2) prove t/SMS.t > >Sponsored-by: Koha-Suomi Oy >--- > C4/Letters.pm | 1 + > C4/SMS.pm | 1 + > t/SMS.t | 1 + > 3 files changed, 3 insertions(+) > >diff --git a/C4/Letters.pm b/C4/Letters.pm >index 7c136f32f9..468fc8f820 100644 >--- a/C4/Letters.pm >+++ b/C4/Letters.pm >@@ -1519,6 +1519,7 @@ sub _send_message_by_sms { > { > destination => $patron->smsalertnumber, > message => $message->{'content'}, >+ message_id => $message->{'message_id'}, > } > ); > >diff --git a/C4/SMS.pm b/C4/SMS.pm >index ce3358f66a..2286ee10a4 100644 >--- a/C4/SMS.pm >+++ b/C4/SMS.pm >@@ -120,6 +120,7 @@ sub send_sms { > $sent = $sender->send_sms( > to => $params->{destination}, > text => $params->{message}, >+ _message_id => $params->{'message_id'}, > ); > }; > >diff --git a/t/SMS.t b/t/SMS.t >index bf8de65152..2e020c05d9 100755 >--- a/t/SMS.t >+++ b/t/SMS.t >@@ -57,6 +57,7 @@ $send_sms = C4::SMS->send_sms({ > destination => '+33123456789', > message => 'my message', > driver => 'Test', >+ message_id => 1, > }); > is( $send_sms, 1, 'send_sms returns 1' ); > >-- >2.25.1 >
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 31481
:
139938
|
140507
|
158114
|
160986
|
167206
|
167330