]> git.koha-community.org Git - koha.git/commit
Bug 18478 - Some notices sent via SMS gateway fail
authorNick Clemens <nick@bywatersolutions.com>
Wed, 17 May 2017 16:54:44 +0000 (12:54 -0400)
committerKatrin Fischer <katrin.fischer.83@web.de>
Mon, 5 Jun 2017 13:35:02 +0000 (15:35 +0200)
commit5c7bee6c4b924ee2d169b7e9b3045701b63e425a
treeb9edb470b4649cb73e71c2b4dbdc54441634d67d
parent7bd3424847c7d4c922a623417636837ab11a6ca2
Bug 18478 - Some notices sent via SMS gateway fail

It seems that for HOLD and DUE (and maybe more) notices we rely on
C4::Letters::SendQueuedMessages
to populate the correct address.

This patch adjust that subroutine to correctly populate the field and/or
fail messages if no SMS provider available

To test:
 1 - Define a messaging prefs for a patron to recieve hold notices via
 SMS
 2 - Ensure you have defined an SMS message for 'HOLD' letter
 3 - Set an SMS alert number for patron
 4 - Set the SMS::Send driver to 'Email'
 5 - Fill a hold for the patron
 6 - Check the db and note the address is null
 7 - run process_message_queue.pl
 8 - Check db - address is null and message pending
 9 - Apply patch
10 - run process_message_queue
11 - Message to_address should be populated and message sent

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 4fa3df9462eaa4a6a2399af4d7036d2f3ee990ce)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
C4/Letters.pm