Attachment 157801 Details for Bug 35148 – Bug 35148: before_send_messages plugin hook does not pass the --where option

[patch] Bug 35148: before_send_messages plugin hook does not pass the --where option

Bug-35148-beforesendmessages-plugin-hook-does-not-.patch (text/plain), 1.58 KB, created by Kyle M Hall (khall) on 2023-10-25 13:43:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2023-10-25 13:43:41 UTC

Size: 1.58 KB

patch

obsolete

>From 7d19fa17c169b60a46935446c91977f50a7c5adf Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 25 Oct 2023 09:34:53 -0400
>Subject: [PATCH] Bug 35148: before_send_messages plugin hook does not pass the
> --where option
>
>The before_send_messages plugin hook currently sends all the non-smtp related options with the exception of the --where option. This should be added.
>
>Test Plan:
>1) Install the latest version of the kitchen sink plugin (
>   https://github.com/bywatersolutions/dev-koha-plugin-kitchen-sink/releases
>   )
>2) Apply this patch
>3) Run process_message_queue.pl with a --where option
>4) Run the command: ./misc/cronjobs/process_message_queue.pl --where "letter_code='test'"
>5) Note the output looks like:
>Plugin hook before_send_message called with the params: $VAR1 = {
>          'verbose' => 0,
>          'where' => 'letter_code=\'test\'',
>          'letter_code' => [],
>          'type' => [],
>          'limit' => undef
>        };
>---
> misc/cronjobs/process_message_queue.pl | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/misc/cronjobs/process_message_queue.pl b/misc/cronjobs/process_message_queue.pl
>index 797567c1c2b..316e5bae1af 100755
>--- a/misc/cronjobs/process_message_queue.pl
>+++ b/misc/cronjobs/process_message_queue.pl
>@@ -100,6 +100,7 @@ if ( C4::Context->config("enable_plugins") ) {
>                         limit       => $limit,
>                         type        => \@type,
>                         letter_code => \@letter_code,
>+                        where       => $where,
>                     }
>                 );
>             }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35148: 157801 | 157802