From 00db0fe1aca297dc04ae70b029c9112cc2a9f849 Mon Sep 17 00:00:00 2001 From: Your Full Name Date: Thu, 6 Sep 2018 13:23:17 +0000 Subject: [PATCH] Bug 21322 - process_message_queue.pl --type should take an argument The help for this script says: -t --type: If supplied, only processes this type of message ( email, sms ) Currently, the type argument is set up wrong, so it does not look for an argument. This patch fixes that. To test, run this command (should work in kohadevbox) or something similar: $ sudo koha-shell -c "perl \ /home/vagrant/kohaclone/misc/cronjobs/process_message_queue.pl -v \ --type=sms" kohadev This should give the following error: "Option type does not take an argument". Apply the patch and run the same command again. This should not give an error. --- misc/cronjobs/process_message_queue.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/misc/cronjobs/process_message_queue.pl b/misc/cronjobs/process_message_queue.pl index 93f5c35..3e0ef51 100755 --- a/misc/cronjobs/process_message_queue.pl +++ b/misc/cronjobs/process_message_queue.pl @@ -44,7 +44,7 @@ GetOptions( 'm|method:s' => \$method, 'h|help|?' => \$help, 'v|verbose' => \$verbose, - 't|type' => \$type, + 't|type:s' => \$type, ); my $usage = << 'ENDUSAGE'; -- 2.1.4