Bug 23173: (follow-up) Fix QA script errors
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 1 Apr 2020 19:13:06 +0000 (15:13 -0400)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 6 Apr 2020 10:05:14 +0000 (11:05 +0100)
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Koha/Illrequest/Availability.pm
koha-tmpl/intranet-tmpl/prog/en/includes/ill-availability-table.inc
koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
koha-tmpl/opac-tmpl/bootstrap/en/includes/ill-availability-table.inc
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt

index a9ca6ef..9013c17 100644 (file)
@@ -4,18 +4,18 @@ package Koha::Illrequest::Availability;
 #
 # This file is part of Koha.
 #
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 3 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 use Modern::Perl;
 
index 69e6679..e609b6b 100644 (file)
@@ -1,6 +1,6 @@
 <div>
-    <table class="ill-availability" id="[% service.id %]">
-        <thead id="[% service.id %]-header">
+    <table class="ill-availability" id="[% service.id | html %]">
+        <thead id="[% service.id | html %]-header">
             <tr>
                 <th>Source</th>
                 <th>Title</th>
@@ -10,7 +10,7 @@
                 <th>Date</th>
             </tr>
         </thead>
-        <tbody id="[% service.id %]-body">
+        <tbody id="[% service.id | html %]-body">
         </tbody>
     </table>
 </div>
index 3976860..deebddf 100644 (file)
                                         </div>
                                         <div class="modal-body">
                                             [% FOR service IN services %]
-                                                <h4 class="ill_availability_sourcename">[% service.name %]</h4>
+                                                <h4 class="ill_availability_sourcename">[% service.name | html %]</h4>
                                                 [% INCLUDE 'ill-availability-table.inc' service=service %]
                                             [% END %]
                                             <span id="service_id_restrict" data-service_id_restrict_plugin="ILL availability - z39.50" data-service_id_restrict_ids=""></span>
                             [% custom_values = whole.custom_value.split('\0') %]
                             [% i = 0 %]
                             [% FOREACH custom_key IN custom_keys %]
-                                <input type="hidden" name="custom_key" value="[% custom_key %]">
-                                <input type="hidden" name="custom_value" value="[% custom_values.$i %]">
+                                <input type="hidden" name="custom_key" value="[% custom_key | html %]">
+                                <input type="hidden" name="custom_value" value="[% custom_values.$i | html %]">
                             [% i = i + 1 %]
                             [% END %]
                             <input type="hidden" name="method" value="create">
                             </div>
                         </form>
                         [% FOR service IN services %]
-                            <h4 class="ill_availability_sourcename">[% service.name %]</h4>
+                            <h4 class="ill_availability_sourcename">[% service.name | html %]</h4>
                             [% INCLUDE 'ill-availability-table.inc' service=service %]
                         [% END %]
                     </div>
index a2597bc..bd99743 100644 (file)
@@ -1,7 +1,7 @@
 <div>
-    <div>[% service.name %]</div>
-    <table class="ill-availability table table-bordered table-striped" id="[% service.id %]">
-        <thead id="[% service.id %]-header">
+    <div>[% service.name | html %]</div>
+    <table class="ill-availability table table-bordered table-striped" id="[% service.id | html %]">
+        <thead id="[% service.id | html %]-header">
             <tr>
                 <th>Source</th>
                 <th>Title</th>
@@ -11,7 +11,7 @@
                 <th>Date</th>
             </tr>
         </thead>
-        <tbody id="[% service.id %]-body">
+        <tbody id="[% service.id | html %]-body">
         </tbody>
     </table>
 </div>
index 77fa2aa..b9611a1 100644 (file)
                                 [% custom_values = whole.custom_value.split('\0') %]
                                 [% i = 0 %]
                                 [% FOREACH custom_key IN custom_keys %]
-                                    <input type="hidden" name="custom_key" value="[% custom_key %]">
-                                    <input type="hidden" name="custom_value" value="[% custom_values.$i %]">
+                                    <input type="hidden" name="custom_key" value="[% custom_key | html %]">
+                                    <input type="hidden" name="custom_value" value="[% custom_values.$i | html %]">
                                 [% i = i + 1 %]
                                 [% END %]
                                 <input type="hidden" name="checked_availability" value="1">
                                 </div>
                             </form>
                             [% FOR service IN services %]
-                                <h4 class="ill_availability_sourcename">[% service.name %]</h4>
+                                <h4 class="ill_availability_sourcename">[% service.name | html %]</h4>
                                 [% INCLUDE 'ill-availability-table.inc' service=service %]
                             [% END %]
                         </div>