From dc92b692e2704c64d1700e7c4fcbf667f6b1402e Mon Sep 17 00:00:00 2001 From: David Cook Date: Tue, 13 Sep 2022 06:46:46 +0000 Subject: [PATCH] Bug 28553: (QA follow-up) default to auto renew email when cron switched used This patch defaults to using "email" rather than "sms" notices when AutoRenewalNotices is set to use the --send-notices cron switch Signed-off-by: David Cook Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 88f63e01d29917e9323c57be97aaa91b9e00fd92) Signed-off-by: Lucas Gass (cherry picked from commit 681730ee1af9ad167f9526903eee6b6681d7d9ff) Signed-off-by: Arthur Suzuki --- misc/cronjobs/automatic_renewals.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/misc/cronjobs/automatic_renewals.pl b/misc/cronjobs/automatic_renewals.pl index 923015a1fa..4d9ff5cdd0 100755 --- a/misc/cronjobs/automatic_renewals.pl +++ b/misc/cronjobs/automatic_renewals.pl @@ -235,7 +235,7 @@ if ( $send_notices && $confirm ) { for my $issue ( @{ $report{$borrowernumber} } ) { my $item = $issue->item; # Force sending of email and only email if pref is set to "cron" - my @transports = $send_notices_pref eq 'preferences' ? keys %{ $borrower_preferences->{'transports'} } : 'sms'; + my @transports = $send_notices_pref eq 'preferences' ? keys %{ $borrower_preferences->{'transports'} } : 'email'; foreach my $transport ( @transports ) { my $letter = C4::Letters::GetPreparedLetter ( module => 'circulation', -- 2.39.5