Bug 22981: Remove type attribute from script tags: Catalog
authorOwen Leonard <oleonard@myacpl.org>
Thu, 23 May 2019 18:35:31 +0000 (18:35 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 30 May 2019 10:26:40 +0000 (10:26 +0000)
This patch removes the "type" attribute from <script> tags in several
catalog-related templates.

To test, apply the patch and confirm that the following work properly
without any JavaScript errors in the browser console:

 - Catalog -> Advanced search
 - Catalog -> Item search
 - Search history

Validating the HTML source of these pages should return no errors
related to the "type" attribute.

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/search-history.tt

index 7bfad35..b41c45f 100644 (file)
 
 [% MACRO jsinclude BLOCK %]
     [% Asset.js("lib/hc-sticky.js") | $raw %]
-    <script type="text/JavaScript">
+    <script>
         /**
          *  Function add_field();
          *  This function allows to display a new field to search.
index c01eab4..1d9e563 100644 (file)
     [% INCLUDE 'datatables.inc' %]
     [% Asset.js("lib/jquery/plugins/jquery.dataTables.columnFilter.js") | $raw %]
     [% Asset.js("lib/hc-sticky.js") | $raw %]
-    <script type="text/javascript">
+    <script>
         var authorised_values = [% authorised_values_json | $raw %];
 
         function loadAuthorisedValuesSelect(select) {
index 7852196..bc109f3 100644 (file)
 [% MACRO jsinclude BLOCK %]
     [% INCLUDE 'datatables.inc' %]
     [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
-    <script type="text/javascript">
-    //<![CDATA[
+    <script>
     $(document).ready(function() {
         // We show table ordered by descending dates by default
         // (so that the more recent query is shown first)