Attachment 40743 Details for Bug 14445 – Bug 14445: Silences warns in letter.tt

[patch] Bug 14445: Silences warns in letter.tt

Bug-14445-Silences-warns-in-lettertt.patch (text/plain), 2.36 KB, created by Aleisha Amohia on 2015-07-01 04:23:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2015-07-01 04:23:36 UTC

Size: 2.36 KB

patch

obsolete

>From 4b400c6affc03ac929846e9ff9057bcee2eafb83 Mon Sep 17 00:00:00 2001
>From: Aleisha <aleishaamohia@hotmail.com>
>Date: Wed, 1 Jul 2015 01:36:00 +0000
>Subject: [PATCH] Bug 14445: Silences warns in letter.tt
>
>When creating a new notice, warn is triggered "Argument "" isn't numeric in numeric gt (>) at line 400". Same warn is triggered when changing Koha module option to any other module.
>
>To test:
>1) Go to Tools, then Notices & Slips
>2) Click 'new notice'. Notice warn in intranet-error.log
>3) Change Koha module to another module. Notice warn is triggered for every change
>4) Apply patch and reload page
>5) Change Koha module to another module. Notice there are no longer warns
>6) Go back to Notices & Slips and click 'new notice' again. Notice there are no warns
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt |    6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>index 3106498..b7dda45 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>@@ -397,14 +397,14 @@ $(document).ready(function() {
>                 <li>
>                   <label for="SQLfieldname_[% letter.message_transport_type %]">Message body:</label>
>                   [% IF letter.message_transport_type == 'sms' %]
>-                    <span id="sms_counter">[% IF letter.content.length > 0 %][% letter.content.length %][% ELSE %]0[% END %]/160 characters</span>
>+                    <span id="sms_counter">[% IF letter.length && letter.length > 0 %][% letter.length %][% ELSE %]0[% END %]/160 characters</span>
>                   [% END %]
>                   <table>
>                     <tr>
>                       <td>
>                         <select name="SQLfieldname" id="SQLfieldname_[% letter.message_transport_type %]" multiple="multiple" size="9">
>-                          [% FOREACH SQLfieldnam IN SQLfieldname %]
>-                            <option value="[% SQLfieldnam.value %]">[% SQLfieldnam.text %]</option>
>+                          [% FOREACH SQLfieldname IN SQLfieldname %]
>+                            <option value="[% SQLfieldname.value %]">[% SQLfieldname.text %]</option>
>                           [% END %]
>                         </select>
>                       </td>
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

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