https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43551
--- Comment #9 from Andrew Fuerste-Henry <[email protected]> --- Created attachment 209228 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=209228&action=edit Bug 43551: Convert process_message_queue.pl to the Getopt::Long::Descriptive convention Standardises the command-line option parsing in misc/cronjobs/process_message_queue.pl on Koha::Script->describe_options (bug 43546), giving it a self-documenting --help and consistent error handling, matching other converted cron scripts. No option was required before and none is required now, so this does not change how the script needs to be run. Test plan: 1. misc/cronjobs/process_message_queue.pl --help Check the description and all options are listed: username, password, type, code, exclude-code, limit, method, verbose, where, exit-on-plugin-failure. 2. Queue up a mix of pending notices (different letter codes and transport types), then run the script with no options and confirm they are all sent as before. 3. Re-run with --code, --exclude-code, --type, --limit, --where and --verbose, individually and combined, and confirm each still filters as documented. 4. Confirm --code and --exclude-code together are still rejected as mutually exclusive. 5. prove t/db_dependent/cronjobs/process_message_queue.pl.t 6. prove t/db_dependent/Letters.t 7. koha-qa.pl passes. Co-Authored-By: Claude Sonnet 5 <[email protected]> Signed-off-by: Juliet Heltibridle <[email protected]> Signed-off-by: Andrew Fuerste Henry <[email protected]> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list -- [email protected] To unsubscribe send an email to [email protected] website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
