From 7eb195b6bdd0bdbf5bfa92a71b051935f78d1db4 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 7 Sep 2023 11:04:32 -0400 Subject: [PATCH] Bug 34731: Throw exception if SendQueuedMessages is passed a bad message_id Test Plan: 1) Apply this patch 2) prove t/db_dependent/Letters.t Signed-off-by: David Nind Signed-off-by: Martin Renvoize Signed-off-by: Tomas Cohen Arazi --- C4/Letters.pm | 13 ++++++++----- t/db_dependent/Letters.t | 18 +++++++++++++++++- 2 files changed, 25 insertions(+), 6 deletions(-) diff --git a/C4/Letters.pm b/C4/Letters.pm index e070697df4..f7efee76be 100644 --- a/C4/Letters.pm +++ b/C4/Letters.pm @@ -25,19 +25,19 @@ use Module::Load::Conditional qw( can_load ); use Try::Tiny; -use C4::Members; use C4::Log qw( logaction ); +use C4::Members; use C4::SMS; use C4::Templates; -use Koha::SMS::Providers; - +use Koha::Auth::TwoFactorAuth; +use Koha::DateUtils qw( dt_from_string output_pref ); use Koha::Email; +use Koha::Exceptions; use Koha::Notice::Messages; use Koha::Notice::Templates; use Koha::Notice::Util; -use Koha::DateUtils qw( dt_from_string output_pref ); -use Koha::Auth::TwoFactorAuth; use Koha::Patrons; +use Koha::SMS::Providers; use Koha::SMTP::Servers; use Koha::Subscriptions; @@ -980,6 +980,9 @@ sub SendQueuedMessages { my $limit = $params->{limit}; my $where = $params->{where}; + Koha::Exceptions::BadParameter->throw("Parameter message_id cannot be empty if passed.") + if ( exists( $params->{message_id} ) && !$params->{message_id} ); + my $smtp_transports = {}; my $count_messages = 0; diff --git a/t/db_dependent/Letters.t b/t/db_dependent/Letters.t index 3d1e34a6bb..94b5f60633 100755 --- a/t/db_dependent/Letters.t +++ b/t/db_dependent/Letters.t @@ -18,7 +18,7 @@ # along with Koha; if not, see . use Modern::Perl; -use Test::More tests => 87; +use Test::More tests => 90; use Test::MockModule; use Test::Warn; use Test::Exception; @@ -158,6 +158,22 @@ my $yesterday = dt_from_string->subtract( days => 1 ); Koha::Notice::Messages->find($messages->[0]->{message_id})->time_queued($yesterday)->store; # SendQueuedMessages + +throws_ok { + C4::Letters::SendQueuedMessages( { message_id => undef } ); +} +'Koha::Exceptions::BadParameter', 'Undef message_id throws an exception'; + +throws_ok { + C4::Letters::SendQueuedMessages( { message_id => 0 } ); +} +'Koha::Exceptions::BadParameter', 'message_id of 0 throws an exception'; + +throws_ok { + C4::Letters::SendQueuedMessages( { message_id => q{} } ); +} +'Koha::Exceptions::BadParameter', 'Empty string message_id throws an exception'; + my $messages_processed = C4::Letters::SendQueuedMessages( { type => 'email' }); is($messages_processed, 0, 'No queued messages processed if type limit passed with unused type'); $messages_processed = C4::Letters::SendQueuedMessages( { type => 'sms' }); -- 2.39.5