]> git.koha-community.org Git - koha.git/commit
Bug 21241: Conditional to choose appropriate message_transport_type for suggestion...
authorAlex Buckley <alexbuckley@catalyst.net.nz>
Thu, 16 Aug 2018 14:06:56 +0000 (02:06 +1200)
committerNick Clemens <nick@bywatersolutions.com>
Mon, 28 Jan 2019 11:42:31 +0000 (11:42 +0000)
commitdfa883bbd00af6c86397d34f3c202e4d9d20d5bf
treec58076dec51865e680cf5bb5d0a0cc47ca8ab6b2
parent358adce2e75bdd317165c86a9b5064bc52d9e2ad
Bug 21241: Conditional to choose appropriate message_transport_type for suggestion notices

Introducing a conditional into C4/Suggestions.pm to check if no patron
email exists but a smsalertnumber does exist then the suggestion notice
message_transport_type is set to 'sms', otherwise the suggestion
message_transport_type='email'.

This ensures patrons with no email address still receieve their
suggestion notices because at present the message_transport_type is
hardcoded to 'email'.

Test plan:
1. Chose a patron who has no email address set, but does have a
smsalertnumber set (this value is set in the Patron messaging
preferences section after the SMSSendDriver syspref is set)

2. Log into the OPAC with that user and submit a suggestion

3. In the staff client go to Acquisitions->Suggestions and tick the
suggestion and set its status to 'Accepted'

4. In the database query the message_queue and notice the
message_transport_type of the message is set to 'email' even though the
patron has no email address set.

5. Apply this patch, restart memcached and plack

6. Repeat steps 2,3 with the same patron and query the message_queue
table again and notice this time the message_transport_type = 'sms'

7. Repeat steps 2,3 with a patron with no smsalertnumber or email
address. Query the database and observe the
message_transport_type=email.

If no smsalertnumber is set then the message_transport_type is set to
'email' by default

Sponsored-By: Brimbank Library, Australia
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Suggestions.pm