From a3f097958cae68e4f3f16c8eb5aa5411d2488bc2 Mon Sep 17 00:00:00 2001 From: Magnus Enger 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. Signed-off-by: Mark Tompsett Signed-off-by: Josef Moravec Signed-off-by: Nick Clemens (cherry picked from commit d74a05d4fe81a599c030024f7944d2bdcca43d12) Signed-off-by: Martin Renvoize --- 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 93f5c35d75..3e0ef5173a 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.39.5