Bug 20761: Fix some js includes in rancor
authorNick Clemens <nick@bywatersolutions.com>
Sun, 13 May 2018 01:44:28 +0000 (01:44 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 29 May 2018 14:16:56 +0000 (14:16 +0000)
To test:
1 - Load the advanced cataloging editor
2 - Inspect the network tab in your browser (F12)
3 - Note failure to load 'cataloging_.js'
4 - Note some other scripts don't have versioning
5 - Apply patch
6 - Reload page
7 - Note cataloging.js loads with version number
8 - Note more js files have versioning

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/editor.tt

index 68a168f..2a0e050 100644 (file)
@@ -1,7 +1,7 @@
-<script src="[% interface %]/lib/codemirror/codemirror-compressed.js"></script>
-<script src="[% interface %]/lib/filesaver.js"></script>
-<script src="[% interface %]/lib/koha/cateditor/marc-mode.js"></script>
-<script src="[% interface %]/lib/require.js"></script>
+[% Asset.js("lib/codemirror/codemirror-compressed.js") %]
+[% Asset.js("lib/filesaver.js") %]
+[% Asset.js("lib/koha/cateditor/marc-mode.js") %]
+[% Asset.js("lib/require.js") %]
 <script>
     var authInfo = {
         [%- FOREACH authtag = authtags -%]
index 5937800..8abcdc5 100644 (file)
@@ -6,8 +6,8 @@
 [% Asset.css("css/datatables.css") %]
 [% Asset.css("lib/codemirror/codemirror.css") %]
 [% Asset.css("css/humanmsg.css") %]
-<script src="[% interface %]/lib/jquery/plugins/humanmsg.js" type="text/javascript"></script>
-<script src="[% interface %]/[% theme %]/js/cataloging_[% KOHA_VERSION %].js" type="text/javascript"></script>
+[% Asset.js("js/cataloging.js") %]
+[% Asset.js("lib/jquery/plugins/humanmsg.js") %]
 [% IF ( bidi ) %]
    [% Asset.css("css/right-to-left.css") %]
 [% END %]