Attachment 96477 Details for Bug 24268 – Bug 24268: Break assignment and warn into separate actions

[patch] Bug 24268: Break assignment and warn into separate actions

Bug-24268-Break-assignment-and-warn-into-separate-.patch (text/plain), 1.22 KB, created by Jonathan Druart on 2019-12-19 06:57:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2019-12-19 06:57:47 UTC

Size: 1.22 KB

patch

obsolete

>From 776cb1726ec7af82fbf25ffa0049de4b46bd432f Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 18 Dec 2019 18:53:04 +0000
>Subject: [PATCH] Bug 24268: Break assignment and warn into separate actions
>
>Signed-off-by: Liz Rea <wizzyrea@gmail.com>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> misc/cronjobs/advance_notices.pl | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/misc/cronjobs/advance_notices.pl b/misc/cronjobs/advance_notices.pl
>index 7c7746cdbf..d8dcf73a51 100755
>--- a/misc/cronjobs/advance_notices.pl
>+++ b/misc/cronjobs/advance_notices.pl
>@@ -582,7 +582,11 @@ sub send_digests {
>                     branchcode     => $branchcode,
>                     message_transport_type => $transport
>                 }
>-            ) || warn "no letter of type '$params->{letter_type}' found for borrowernumber $borrowernumber. Please see sample_notices.sql";
>+            );
>+            unless ( $letter ){
>+                warn "no letter of type '$params->{letter_type}' found for borrowernumber $borrowernumber. Please see sample_notices.sql";
>+                next;
>+            }
>             push @letters, $letter if $letter;
>         }
> 
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24268: 96458 | 96459 | 96462 | 96463 | 96476 | 96477