From ee299e4b863de61ceb6f515ccb21fc4ec43e8a7a Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 30 Mar 2026 12:58:13 +0200 Subject: [PATCH] Bug 42282: Tidy all script tags - catalogue In order to tidy all the script tags of the .tt files we isolate the interpolation of Template::Toolkit variables in a separate script tag. This patch focuses on the catalogue module. The 7 main changes are in the following files: * ISBDdetail.tt * MARCdetail.tt * detail.tt * imageviewer.tt * labeledMARCdetail.tt * moredetail.tt * stockrotation.tt Several identical changes (logged_in_user_borrowernumber, searchid, biblio_id, browser, etc.) detail.tt needs to be tested a bit more thoroughly but the changes are still trivial. Signed-off-by: Paul Derscheid Signed-off-by: Jan Kissig Signed-off-by: Pedro Amorim --- .../prog/en/modules/catalogue/ISBDdetail.tt | 12 +- .../prog/en/modules/catalogue/MARCdetail.tt | 14 +- .../prog/en/modules/catalogue/advsearch.tt | 2 +- .../prog/en/modules/catalogue/detail.tt | 152 ++++++++++-------- .../prog/en/modules/catalogue/imageviewer.tt | 18 ++- .../prog/en/modules/catalogue/issuehistory.tt | 2 +- .../prog/en/modules/catalogue/itemsearch.tt | 2 +- .../en/modules/catalogue/labeledMARCdetail.tt | 18 ++- .../prog/en/modules/catalogue/moredetail.tt | 13 +- .../prog/en/modules/catalogue/results.tt | 2 +- .../prog/en/modules/catalogue/showelastic.tt | 2 +- .../en/modules/catalogue/stockrotation.tt | 7 +- 12 files changed, 144 insertions(+), 100 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/ISBDdetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/ISBDdetail.tt index d08d7ae73a..3474d4f540 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/ISBDdetail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/ISBDdetail.tt @@ -91,15 +91,17 @@ [% END %] [% Asset.js("js/browser.js") | $raw %] + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt index 6a12ccf5cd..6551302abe 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt @@ -215,15 +215,17 @@ [% END %] [% Asset.js("js/browser.js") | $raw %] + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt index b732e9742a..6dcb6ad85b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt @@ -401,7 +401,7 @@ [% Asset.js("js/browser.js") | $raw %] [% Asset.css("css/humanmsg.css") | $raw %] [% Asset.js("lib/jquery/plugins/humanmsg.js") | $raw %] - + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] [% IF ( ( Koha.Preference('CatalogConcerns') || Koha.Preference('OpacCatalogConcerns') ) && CAN_user_editcatalogue_edit_catalogue ) %] - [% IF ( Koha.Preference('NovelistSelectStaffEnabled') && Koha.Preference('NovelistSelectStaffProfile') && ( normalized_isbn || normalized_upc ) ) %] - [% IF Koha.Preference('AcquisitionDetails') %] - @@ -2165,9 +2194,6 @@ [% END %] [% IF ( reviews ) %] - + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt index 3d93fa27b6..24753a2577 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt @@ -602,16 +602,19 @@ [% Asset.js("js/modals/place_booking.js") | $raw %] [% END %] + + [% IF ( Koha.Preference('CatalogConcerns') ) %] - [% Asset.js("js/modals/add_catalog_concern.js") | $raw %] [% END %] + [% END %]