Attachment 112650 Details for Bug 26851 – Bug 26851: Don't generate a notice to library if no content

[patch] Bug 26851: Don't generate a notice to library if no content

Bug-26851-Dont-generate-a-notice-to-library-if-no-.patch (text/plain), 1.48 KB, created by Nick Clemens (kidclamp) on 2020-10-28 19:07:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2020-10-28 19:07:08 UTC

Size: 1.48 KB

patch

obsolete

>From fb2f15e6d746bf5f57af2b8c3d23147c269a24fb Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 28 Oct 2020 19:00:28 +0000
>Subject: [PATCH] Bug 26851: Don't generate a notice to library if no content
>
>To test:
> 1 - Check out an over due to a patron
> 2 - Set overdue triggers to a trigger less than number of days overdue
> 3 - Make sure the patron has no email
> 4 - Set content of ODUE notice to:
>    [%# Comment will not print %]
> 5 - perl misc/cronjobs/overdue_notices.pl
> 6 - Check the message queue
> 7 - There is a notice sent to the library:
>    "These messages were not sent directly to the patrons."
>    With no messages
> 8 - Apply patch
> 9 - Repeat
>10 - No new message generated
>---
> misc/cronjobs/overdue_notices.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl
>index 591feea145..70c444ee70 100755
>--- a/misc/cronjobs/overdue_notices.pl
>+++ b/misc/cronjobs/overdue_notices.pl
>@@ -705,7 +705,7 @@ END_SQL
>                             message_transport_type => $letter_exists ? $effective_mtt : $mtt,
>                         }
>                     );
>-                    unless ($letter) {
>+                    unless ($letter && $letter->{content}) {
>                         $verbose and warn qq|Message '$overdue_rules->{"letter$i"}' content not found|;
>                         # this transport doesn't have a configured notice, so try another
>                         next;
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26851: 112650 | 114371 | 115016