]> git.koha-community.org Git - koha.git/commit
Bug 18457 - process_message_queue.pl will die if a patron has no sms_provider_id...
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 19 Apr 2017 13:52:38 +0000 (09:52 -0400)
committerKatrin Fischer <katrin.fischer.83@web.de>
Sat, 13 May 2017 11:45:38 +0000 (13:45 +0200)
commitd9fa24e8bb9f43982773e1a79d629d2e8a2cbf2f
tree2dba435d7c13d3b38430db00a1734163eebf8450
parent2695be154fdf9b3fc8b678fbcd7f62a2e139e2f9
Bug 18457 - process_message_queue.pl will die if a patron has no sms_provider_id set but sms via email is enabled for that patron

If SMS via Email is enabled, and a patron has opted for SMS messages, but has not selected a service provider, the cronjob will die with the error
Can't call method "domain" on an undefined value at /usr/share/koha/lib/C4/Letters.pm line 1055.
This will cause all messages that come after the error to not be sent!

Test Plan:
1) Enable SMS via Email
2) Enable SMS for a patron, but don't set a provider
3) Perform an action that will trigger an sms message to go into
   the holds queue ( item due, item checkout, etc )
4) Run process_message_queue.pl, note the error
5) Apply the patch
4) Run process_message_queue.pl, no error this time!

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit c7541091741878d28f648df8681a691cf787334c)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
C4/Letters.pm