Browse Source

Bug 23227: Remove type attribute from script tags: Reports

This patch removes the "type" attribute from <script> tags in two
reports templates.

To test, apply the patch and confirm that examples of affected pages
work properly without any JavaScript errors in the browser console:

 - Reports -> Dictionary
   - Delete
   - Specifying a date range for a date column
 - Reports -> Items lost
   - Report results
     -> Datatables functionality
     -> Selection tools for export

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

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: frederik chenier <frederik.chenier@inlibro.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
remotes/origin/19.11.x
Owen Leonard 5 years ago
committed by Martin Renvoize
parent
commit
d88709ec2d
Signed by: martin.renvoize GPG Key ID: 422B469130441A0F
  1. 2
      koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt
  2. 2
      koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt

2
koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt

@ -273,7 +273,7 @@
[% MACRO jsinclude BLOCK %]
[% INCLUDE 'calendar.inc' %]
<script type="text/javascript">
<script>
var MSG_CONFIRM_DELETE = _("Are you sure you want to delete this dictionary definition? This cannot be undone.");
$(document).ready(function() {

2
koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt

@ -192,7 +192,7 @@
[% INCLUDE 'columns_settings.inc' %]
[% Asset.js("js/table_filters.js") | $raw %]
[% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
<script type='text/javascript'>
<script>
$(document).ready(function() {
var columns_settings = [% ColumnsSettings.GetColumns( 'reports', 'lostitems', 'lostitems-table', 'json' ) | $raw %];
var lostitems_table = KohaTable("lostitems-table", {

Loading…
Cancel
Save