Attachment 40856 Details for Bug 14445 – Bug 14445: Silences warn in letter.pl

[patch] Bug 14445: Silences warn in letter.pl

Bug-14445-Silences-warn-in-letterpl.patch (text/plain), 1.30 KB, created by Jonathan Druart on 2015-07-09 07:42:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-07-09 07:42:31 UTC

Size: 1.30 KB

patch

obsolete

>From ed80d91b0ed178e7e3a0ecff4f68d694cb2d4b10 Mon Sep 17 00:00:00 2001
>From: Aleisha <aleishaamohia@hotmail.com>
>Date: Wed, 24 Jun 2015 01:15:32 +0000
>Subject: [PATCH] Bug 14445: Silences warn in letter.pl
>
>When changing Koha module to 'Circulation', there is a warn saying that $code is uninitialized. This patch sets $code to an empty string to silence the warn.
>
>To test:
>1) Go to Tools, the Notices & Slips
>2) Click 'new notice' (This will trigger warns, but ignore these as they will be corrected in the next patch)
>3) Change Koha module to 'Circulation'
>4) Notice warn about uninitialized $code variable
>5) Apply patch and reload page, change Koha module to 'Circulation'
>6) Notice page still works and warns are gone
>
>Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
>---
> tools/letter.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/tools/letter.pl b/tools/letter.pl
>index ca9f0ce..8ef07be 100755
>--- a/tools/letter.pl
>+++ b/tools/letter.pl
>@@ -227,7 +227,7 @@ sub add_form {
> 
>         }
> 
>-        if ( $module eq 'circulation' && $code eq "CHECKIN" ) {
>+        if ( $module eq 'circulation' and $code and $code eq "CHECKIN" ) {
>             push @{$field_selection}, add_fields('old_issues');
>         } else {
>             push @{$field_selection}, add_fields('issues');
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 14445: 40545 | 40743 | 40856 | 40857 | 40922 | 40923