Bug 27561: Remove type attribute from script tags: Various templates

This patch replaces remaining instances of <script type="javascript"> in templates with "<script>." To test, apply the patch and check the changes to the template. Verify that the changes look correct. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
2021-01-27 12:48:57 +00:00 · 2021-01-27 12:48:57 +00:00 · 5c68609110
commit 5c68609110
parent ef5ec1731a
17 changed files with 18 additions and 18 deletions
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
@ -872,7 +872,7 @@
 </div>

 [% MACRO jsinclude BLOCK %]
-    <script type="text/javascript">
+    <script>
         $(document).ready(function() {
            $('#abouttabs').tabs();
         });
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt
@ -150,7 +150,7 @@
 [% MACRO jsinclude BLOCK %]
    [% Asset.js("js/desk_selection.js") | $raw %]
    [% Asset.js("js/register_selection.js") | $raw %]
-    <script type="text/javascript">
+    <script>
        $(document).ready( function() {
            if ( document.location.hash ) {
                $( '#loginform' ).append( '<input name="auth_forwarded_hash" type="hidden" value="' + document.location.hash + '"/>' );
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/patron-clubs-tab.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/patron-clubs-tab.tt
@ -64,7 +64,7 @@
 [% END %]

 [% IF CAN_user_clubs_enroll %]
-<script type="text/javascript">
+<script>
 function loadEnrollmentForm( id ) {
    $("body").css("cursor", "progress");
    $('#clubs-tab').load('/cgi-bin/koha/clubs/patron-enroll.pl?borrowernumber=[% borrowernumber | html %]&id=' + id, function() {
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/patron-enroll.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/patron-enroll.tt
@ -36,7 +36,7 @@
    </form>
 </div>

-<script type="text/javascript">
+<script>
 function addEnrollment() {
    $("body").css("cursor", "progress");
    $.ajax({
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt
@ -119,7 +119,7 @@
 [% MACRO jsinclude BLOCK %]
    [% INCLUDE 'datatables.inc' %]

-    <script type="text/javascript">
+    <script>
        var search = 1;
        $(document).ready(function(){
            $("#info").hide();
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course.tt
@ -156,7 +156,7 @@

 [% IF ( departments ) %]
    [% MACRO jsinclude BLOCK %]
-        <script type="text/javascript">
+        <script>
            $(document).ready(function(){
                $( "#find_instructor" ).autocomplete({
                    source: "/cgi-bin/koha/circ/ysearch.pl",
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt
@ -236,7 +236,7 @@
    </div> <!-- /.container-fluid -->

 [% MACRO jsinclude BLOCK %]
-    <script type="text/javascript">
+    <script>
    //<![CDATA[
    var MSG_CONFIRM_DELETE = _("Are you sure you want to delete this news item? This cannot be undone.");
        $(document).ready(function(){
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt
@ -191,7 +191,7 @@
 [% MACRO jsinclude BLOCK %]
    [% INCLUDE 'str/members-menu.inc' %]
    [% Asset.js("js/members-menu.js") | $raw %]
-    <script type="text/javascript">
+    <script>
        $(function() {
            $( "#pending_updates" ).accordion();
        });
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/update-child.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/update-child.tt
@ -51,7 +51,7 @@

 [% MACRO jsinclude BLOCK %]
    [% INCLUDE 'datatables.inc' %]
-    <script type="text/javascript">
+    <script>

        $(document).ready(function() {
            $("#catst").dataTable($.extend(true, {}, dataTablesDefaults, {
@ -65,7 +65,7 @@
        });
    </script>
    [% IF ( SUCCESS ) %]
-        <script type="text/javascript">
+        <script>
            self.opener.location.href='/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber | html %]';
            window.close();
        </script>
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt
@ -71,7 +71,7 @@
 [% MACRO jsinclude BLOCK %]
    [% Asset.js("js/background-job-progressbar.js") | $raw %]
    [% Asset.js("js/file-upload.js") | $raw %]
-    <script type="text/javascript">
+    <script>
        var xhr;
        $(document).ready(function(){
            $("#enqueuefile").hide();
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/acqui-search-result.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/acqui-search-result.tt
@ -37,7 +37,7 @@
    <div id="closewindow"><a href="#" class="btn btn-default btn-default close">Cancel</a></div>

 [% MACRO jsinclude BLOCK %]
-  <script type="text/javascript">
+  <script>
    function GetIt(aqbooksellerid,name){
      opener.document.f.aqbooksellerid.value = aqbooksellerid;
      opener.document.f.aqbooksellername.value = name;
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tt
@ -62,7 +62,7 @@
 <div id="closewindow"><a class="btn btn-default btn-default close" href="#">Close</a></div>

 [% MACRO jsinclude BLOCK %]
-    <script type="text/javascript">
+    <script>
        $(document).ready(function(){
            $(".select_title").on("click",function(e){
                e.preventDefault();
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/routing-preview-slip.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/routing-preview-slip.tt
@ -39,7 +39,7 @@
    <div id="closewindow" class="noprint"><a class="btn btn-default btn-default" id="print_slip" href="#"><i class="fa fa-print"></i> Print</a> <a class="btn btn-default btn-default close" href="#">Close</a></div>

 [% MACRO jsinclude BLOCK %]
-    <script type="text/javascript">
+    <script>
        $(document).ready(function(){
            $("#print_slip").on("click",function(e){
                e.preventDefault();
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-renew.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-renew.tt
@ -79,7 +79,7 @@

 [% MACRO jsinclude BLOCK %]
    [% INCLUDE 'calendar.inc' %]
-    <script type="text/javascript">
+    <script>
        $(document).ready(function(){
            $(".close").on("click", function(e){
                e.preventDefault();
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt
@ -299,7 +299,7 @@
    [% INCLUDE 'datatables.inc' %]
    [% Asset.js("js/pages/batchMod.js") | $raw %]
    [% Asset.js("js/browser.js") | $raw %]
-    <script type="text/javascript">
+    <script>
        // Prepare array of all column headers, incrementing each index by
        // two to accommodate control and title columns
        var allColumns = new Array([% FOREACH item_header_loo IN item_header_loop %]'[% loop.count | html %]'[% UNLESS ( loop.last ) %],[% END %][% END %]);
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-edit.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-edit.tt
@ -9,7 +9,7 @@
 [% Asset.js("js/cataloging.js") | $raw %]
 [% INCLUDE 'datatables.inc' %]
 [% Asset.js("js/pages/batchMod.js") | $raw %]
-<script type="text/javascript">
+<script>
 //<![CDATA[

 // Prepare array of all column headers, incrementing each index by
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
@ -290,7 +290,7 @@
    [% Asset.js("js/tools-menu.js") | $raw %]
    [% INCLUDE 'datatables.inc' %]
    [% INCLUDE 'calendar.inc' %]
-    <script type="text/javascript">
+    <script>
        function checkForm() {
            if ( $('#uploadbarcodes').val() && $('#barcodelist').val() ) {
                alert(_("You have uploaded a barcode file and scanned barcodes at the same time. Please choose one of the two options."));