Bug 25097

Summary: Add option to message_queue to allow for only specific sending notices
Product: Koha Reporter: Joy Nelson <1joynelson>
Component: NoticesAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: aleisha, alex.arnaud, fiona.borthwick, jonathan.druart, lucas, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
This adds a new command line option -c|--code to the process_message_queue.pl cron job allowing to pick which letter codes will be processed and send when the job runs. This will allow to send different notices at different times depending on your cron job setup.
Version(s) released in:
20.11.00, 20.05.02
Circulation function:
Bug Depends on:    
Bug Blocks: 27096, 27265    
Attachments: Bug 25097: Add option to message_queue to allow for only specific sending notices
Bug 25097: Add option to message_queue to allow for only specific sending notices
Bug 25097: Add option to message_queue to allow for only specific sending notices

Description Joy Nelson 2020-04-09 14:45:41 UTC
The message_queue.pl currently will send all the messages queued up.  It would be nice to have some control over which messages are to be sent.  

Example: We have libraries who only wanted to send 'hold notices' but not overdue, advance notice, etc during the covid-19 quarantine.  And some that were the opposite - turn off hold notices, but send other generated notices.
Comment 1 Kyle M Hall (khall) 2020-04-09 15:27:58 UTC
Created attachment 102646 [details] [review]
Bug 25097: Add option to message_queue to allow for only specific sending notices

The message_queue.pl currently will send all the messages queued up.  It would be nice to have some control over which messages are to be sent.

Example: We have libraries who only wanted to send 'hold notices' but not overdue, advance notice, etc during the covid-19 quarantine.  And some that were the opposite - turn off hold notices, but send other generated notices.

Test Plan:
1) Apply this patch
2) Enqueue some messages with different letter codes
3) Run process_message_queue.pl with the new -c parameter using one of the letter codes enqueued
4) Note that only the messages with the selected code were processed!
Comment 2 Andrew Fuerste-Henry 2020-04-09 15:52:48 UTC
Created attachment 102649 [details] [review]
Bug 25097: Add option to message_queue to allow for only specific sending notices

The message_queue.pl currently will send all the messages queued up.  It would be nice to have some control over which messages are to be sent.

Example: We have libraries who only wanted to send 'hold notices' but not overdue, advance notice, etc during the covid-19 quarantine.  And some that were the opposite - turn off hold notices, but send other generated notices.

Test Plan:
1) Apply this patch
2) Enqueue some messages with different letter codes
3) Run process_message_queue.pl with the new -c parameter using one of the letter codes enqueued
4) Note that only the messages with the selected code were processed!

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Comment 3 Alex Arnaud 2020-06-19 14:48:14 UTC
Created attachment 106101 [details] [review]
Bug 25097: Add option to message_queue to allow for only specific sending notices

The message_queue.pl currently will send all the messages queued up.  It would be nice to have some control over which messages are to be sent.

Example: We have libraries who only wanted to send 'hold notices' but not overdue, advance notice, etc during the covid-19 quarantine.  And some that were the opposite - turn off hold notices, but send other generated notices.

Test Plan:
1) Apply this patch
2) Enqueue some messages with different letter codes
3) Run process_message_queue.pl with the new -c parameter using one of the letter codes enqueued
4) Note that only the messages with the selected code were processed!

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Alex Arnaud <alex.arnaud@biblibre.com>
Comment 4 Jonathan Druart 2020-06-24 13:31:52 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 5 Lucas Gass (lukeg) 2020-07-13 14:34:32 UTC
backported to 20.05.x for 20.05.02
Comment 6 Aleisha Amohia 2020-07-16 03:55:37 UTC
enhancement, not backported to 19.11.x