From 7603e276c76152479df8899ade38ccbb50bdeb39 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 25 Oct 2018 12:55:43 -0300 Subject: [PATCH] Bug 21669: Do not html filter TT assignement statements Why? Because we must filter the variables when we display them. If we escape them on assignement, they will be double escaped: [% XXX = "pouet" | html %] [% XXX | html %] => <span>pouet</span> Also it will bring trouble if we are assigning a structure (see bug 21663 for instance). --- .../en/includes/admin-items-search-field-form.inc | 2 +- .../prog/en/includes/branch-selector.inc | 4 ++-- koha-tmpl/intranet-tmpl/prog/en/modules/about.tt | 4 ++-- .../intranet-tmpl/prog/en/modules/acqui/basket.tt | 16 ++++++++-------- .../prog/en/modules/acqui/booksellers.tt | 2 +- .../intranet-tmpl/prog/en/modules/acqui/invoice.tt | 8 ++++---- .../prog/en/modules/acqui/neworderempty.tt | 6 +++--- .../prog/en/modules/admin/authorised_values.tt | 4 ++-- .../prog/en/modules/admin/categories.tt | 4 ++-- .../prog/en/modules/admin/matching-rules.tt | 2 +- .../prog/en/modules/admin/patron-attr-types.tt | 4 ++-- .../prog/en/modules/catalogue/detail.tt | 4 ++-- .../prog/en/modules/catalogue/itemsearch.tt | 4 ++-- .../prog/en/modules/catalogue/results.tt | 2 +- .../prog/en/modules/cataloguing/merge.tt | 10 +++++----- .../prog/en/modules/circ/circulation.tt | 4 ++-- .../en/modules/circ/circulation_batch_checkouts.tt | 4 ++-- .../prog/en/modules/ill/ill-requests.tt | 2 +- .../prog/en/modules/members/discharge.tt | 4 ++-- .../prog/en/modules/members/memberentrygen.tt | 2 +- .../prog/en/modules/members/members-update.tt | 2 +- .../prog/en/modules/reports/cash_register_stats.tt | 4 ++-- .../prog/en/modules/reports/guided_reports_start.tt | 2 +- .../intranet-tmpl/prog/en/modules/tools/letter.tt | 2 +- .../prog/en/modules/tools/modborrowers.tt | 2 +- .../opac-tmpl/bootstrap/en/includes/item-status.inc | 6 +++--- .../bootstrap/en/includes/opac-topissues.inc | 2 +- .../opac-tmpl/bootstrap/en/modules/opac-detail.tt | 20 ++++++++++---------- .../bootstrap/en/modules/opac-illrequests.tt | 4 ++-- .../bootstrap/en/modules/opac-opensearch.tt | 2 +- .../bootstrap/en/modules/opac-readingrecord.tt | 10 +++++----- .../opac-tmpl/bootstrap/en/modules/opac-results.tt | 6 +++--- .../opac-tmpl/bootstrap/en/modules/opac-shelves.tt | 4 ++-- .../bootstrap/en/modules/opac-showreviews-rss.tt | 2 +- .../bootstrap/en/modules/opac-showreviews.tt | 2 +- .../opac-tmpl/bootstrap/en/modules/opac-user.tt | 8 ++++---- 36 files changed, 85 insertions(+), 85 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/admin-items-search-field-form.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/admin-items-search-field-form.inc index 454bd113f1..aaeef9045c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/admin-items-search-field-form.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/admin-items-search-field-form.inc @@ -37,7 +37,7 @@ [% adjustment.adjustment_id | html %] - [% reasons = AuthorisedValues.Get("ADJ_REASON") | html %] + [% reasons = AuthorisedValues.Get("ADJ_REASON") %] [% IF reasons.0 %] - [% reasons = AuthorisedValues.Get("ADJ_REASON") | html %] + [% reasons = AuthorisedValues.Get("ADJ_REASON") %] [% IF reasons.0 %]
  • diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt index ee4b764741..a9f0c63972 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt @@ -327,17 +327,17 @@ [% Budget_name | html %] [% ELSE %] - [% active_count = 0 | html %] + [% active_count = 0 %] [% IF !ordernumber %] [% FOREACH budget_loo IN budget_loop %] - [% active_count= active_count + budget_loo.b_active | html %] + [% active_count= active_count + budget_loo.b_active %] [% END %] [% END %] [% PROCESS norms_options selected_norm=selected_norm %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt index 76c038f24f..0ad7467271 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt @@ -260,9 +260,9 @@ [% item.description | html %] [% IF ( item.branches && item.branches.size > 0 ) %] - [% branches_str = "" | html %] + [% branches_str = "" %] [% FOREACH branch IN item.branches %] - [% branches_str = branches_str _ " " _ branch.branchname _ "(" _ branch.branchcode _ ")" | html %] + [% branches_str = branches_str _ " " _ branch.branchname _ "(" _ branch.branchcode _ ")" %] [% END %] [% IF item.branches.size > 1 %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt index 451c10526f..06f1df0d54 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -8,9 +8,9 @@ [% USE ColumnsSettings %] [% IF Koha.Preference('AmazonAssocTag') %] - [% AmazonAssocTag = '?tag=' _ Koha.Preference('AmazonAssocTag') | html %] + [% AmazonAssocTag = '?tag=' _ Koha.Preference('AmazonAssocTag') %] [% ELSE %] - [% AmazonAssocTag = '' | html %] + [% AmazonAssocTag = '' %] [% END %] [% ShowCourseReserves = 0 | html %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt index 05afac4f69..a7abe0f9f7 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt @@ -72,9 +72,9 @@ [% IF items_search_fields.size %] [% FOREACH field IN items_search_fields %] - [% marcfield = field.tagfield | html %] + [% marcfield = field.tagfield %] [% IF field.tagsubfield %] - [% marcfield = marcfield _ '$' _ field.tagsubfield | html %] + [% marcfield = marcfield _ '$' _ field.tagsubfield %] [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt index ce5179f888..c49d8b3894 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt @@ -399,7 +399,7 @@ [% SEARCH_RESULT.XSLTResultsRecord | $raw %] [% ELSE %]

    [% SEARCH_RESULT.result_number | html %]. - [% biblionumber = SEARCH_RESULT.biblionumber | html %] + [% biblionumber = SEARCH_RESULT.biblionumber %] [% INCLUDE 'biblio-default-view.inc' %] [% IF ( SEARCH_RESULT.title ) %][% SEARCH_RESULT.title | html %][% ELSE %]No title[% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt index d7d378a805..80c8cc573e 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt @@ -46,12 +46,12 @@ div#result { margin-top: 1em; } Biblionumber [% FOREACH key IN report_header.keys.sort %] - [% tag = key.substr(0, 3) | html %] - [% code = key.substr(3, 1) | html %] + [% tag = key.substr(0, 3) %] + [% code = key.substr(3, 1) %] [% IF code == '@' %] - [% header = tag | html %] + [% header = tag %] [% ELSE %] - [% header = tag _ '$' _ code | html %] + [% header = tag _ '$' _ code %] [% END %] [% header | html %] [% END %] @@ -68,7 +68,7 @@ div#result { margin-top: 1em; } [% FOREACH key IN report_header.keys.sort %] - [% values = record.fields.$key | html %] + [% values = record.fields.$key %] [% IF values %] [% FOREACH value IN record.fields.$key %] [% value | html %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt index 2b273e047a..ee23e4e24d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -155,7 +155,7 @@ [% IF ( itemtype_notforloan ) %] Item type is normally not for loan. [% ELSIF ( item_notforloan ) %] - [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, item_notforloan, 0 ) | html %] + [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, item_notforloan, 0 ) %] Item is normally not for loan [% IF (item_notforloan_lib) %]([% item_notforloan_lib | html %])[% END %]. [% END %] [% IF CAN_user_circulate_force_checkout %] @@ -341,7 +341,7 @@ [% IF ( itemtype_notforloan ) %] Item type not for loan. [% ELSIF ( item_notforloan ) %] - [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, item_notforloan, 0 ) | html %] + [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, item_notforloan, 0 ) %] Item not for loan [% IF (item_notforloan_lib) %]([% item_notforloan_lib | html %])[% END %]. [% END %]

  • diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt index eada7334a2..566dfdffaa 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt @@ -152,7 +152,7 @@ [% IF checkout_info.itemtype_notforloan %]
  • Item type is normally not for loan.
  • [% ELSIF checkout_info.item_notforloan %] - [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, checkout_info.item_notforloan, 0 ) | html %] + [% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, checkout_info.item_notforloan, 0 ) %]
  • Item is normally not for loan [% IF item_notforloan_lib %]([% item_notforloan_lib | html %])[% END %].
  • [% END %] [% END %] @@ -208,7 +208,7 @@ [% IF checkout_info.itemtype_notforloan %]
  • Item type not for loan.
  • [% ELSIF checkout_info.item_notforloan %] - [% item_notforloan_lib = AuthorisedValues.GetByCode( checkout_info.authvalcode_notforloan, checkout_info.item_notforloan, 0 ) | html %] + [% item_notforloan_lib = AuthorisedValues.GetByCode( checkout_info.authvalcode_notforloan, checkout_info.item_notforloan, 0 ) %]
  • Item not for loan [% IF checkout_info.item_notforloan_lib %]([% checkout_info.item_notforloan_lib | html %])[% END %].
  • [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt index 584616263d..7250f5d697 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt @@ -427,7 +427,7 @@
  • - [% stat = request.status | html %] + [% stat = request.status %] [% request.capabilities.$stat.name | html %]
  • diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/discharge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/discharge.tt index 694f4942ac..d3d636d022 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/discharge.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/discharge.tt @@ -56,8 +56,8 @@ [% FOR d IN validated_discharges %] - [% d.needed | $KohaDates with_hours = 1 | html %] - [% d.validated | $KohaDates with_hours = 1 | html %] + [% d.needed | $KohaDates with_hours = 1 %] + [% d.validated | $KohaDates with_hours = 1 %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt index e45429532a..49ddce8935 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -1304,7 +1304,7 @@ $(document).ready(function () { $("#entryform,#saverecord").hide(); - [% q_add_f = Koha.Preference('PatronQuickAddFields').split('\|') | html %] + [% q_add_f = Koha.Preference('PatronQuickAddFields').split('\|') %] var qaddfields = [[% FOREACH field IN q_add_f.unique %]"[% field | html %]",[% END %]]; var skipped_fields = ["contactname","contactfirstname","relationship"]; //Guarantor form is pulled as a whole, ignore individual fields $("#entryform label").each(function () { diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt index 687afdee93..32677d46fe 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt @@ -99,7 +99,7 @@
    [% FOREACH pm IN PendingModifications %] [% SET borrowernumber = pm.borrowernumber %] -

    [% borrowers.$borrowernumber.surname | html %][% IF ( borrowers.$borrowernumber.firstname ) %], [% borrowers.$borrowernumber.firstname | html %][% END %] - [% Categories.GetName( borrowers.$borrowernumber.categorycode ) | html %] ([% Branches.GetName( borrowers.$borrowernumber.branchcode ) | html %]) ([% borrowers.$borrowernumber.cardnumber | html %]) - Requested on [% pm.timestamp | $KohaDates with_hours = 1 | html %]

    +

    [% borrowers.$borrowernumber.surname | html %][% IF ( borrowers.$borrowernumber.firstname ) %], [% borrowers.$borrowernumber.firstname | html %][% END %] - [% Categories.GetName( borrowers.$borrowernumber.categorycode ) | html %] ([% Branches.GetName( borrowers.$borrowernumber.branchcode ) | html %]) ([% borrowers.$borrowernumber.cardnumber | html %]) - Requested on [% pm.timestamp | $KohaDates with_hours = 1 %]

    diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt index 018cdee0e4..f0439bc99c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt @@ -122,7 +122,7 @@ [% END %] [% FOREACH manualinv IN manualinv_types %] - [% value_manualinv = manualinv.authorised_value|truncate(5, '') | html %] + [% value_manualinv = manualinv.authorised_value|truncate(5, '') %] [% IF transaction_type == value_manualinv %] [% ELSE %] @@ -218,7 +218,7 @@ New card [% ELSE %] [% FOREACH manualinv IN manualinv_types %] - [% value_manualinv = manualinv.authorised_value|truncate(5, '') | html %] + [% value_manualinv = manualinv.authorised_value|truncate(5, '') %] [% IF loopresul.accounttype == value_manualinv %] [% manualinv.authorised_value | html %] [% LAST %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt index f432cf00b0..4aaa49ef64 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt @@ -743,7 +743,7 @@ canned reports and writing custom SQL reports.

    [% FOREACH result IN results %] [% FOREACH cells IN result.cells %] - [% place = loop.index | html %] + [% place = loop.index %] [% IF header_row.$place.cell == 'itemnumber' %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt index 56cca283ed..149385e8e0 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt @@ -91,7 +91,7 @@ [% FOREACH lette IN letter %] - [% can_edit = lette.branchcode || !independant_branch | html %] + [% can_edit = lette.branchcode || !independant_branch %] [% IF lette.branchname %][% lette.branchname | html %][% ELSE %](All libraries)[% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt index 94e80506e7..a79974acfc 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt @@ -69,7 +69,7 @@ [% END %] [% IF ( op == 'show') && (!borrowers) && (!notfoundcardnumbers) # Alert if no patrons given%] - [% op = 'noshow' # Change op to prevent display in code below | html %] + [% op = 'noshow' # Change op to prevent display in code below %]

    Batch patrons modification

    No patron card numbers given.

    diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc index 55474da7a8..99bdeed344 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc @@ -8,7 +8,7 @@ [% IF ( item.itemlost ) %] [% SET itemavailable = 0 %] - [% av_lib_include = AuthorisedValues.GetByCode( 'LOST', item.itemlost, 1 ) | html %] + [% av_lib_include = AuthorisedValues.GetByCode( 'LOST', item.itemlost, 1 ) %] [% IF ( av_lib_include ) %] [% av_lib_include | html %] [% ELSE %] @@ -53,7 +53,7 @@ [% IF ( item.withdrawn ) %] [% SET itemavailable = 0 %] - [% av_lib_include = AuthorisedValues.GetByCode( 'WITHDRAWN', item.withdrawn, 1 ) | html %] + [% av_lib_include = AuthorisedValues.GetByCode( 'WITHDRAWN', item.withdrawn, 1 ) %] [% IF av_lib_include %] [% av_lib_include | html %] [% ELSE %] @@ -76,7 +76,7 @@ [% IF ( item.damaged ) %] [% SET itemavailable = 0 %] - [% av_lib_include = AuthorisedValues.GetByCode( 'DAMAGED', item.damaged, 1 ) | html %] + [% av_lib_include = AuthorisedValues.GetByCode( 'DAMAGED', item.damaged, 1 ) %] [% IF av_lib_include %] [% av_lib_include | html %] [% ELSE %] diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-topissues.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-topissues.inc index 5bf505b7d2..6f08f1cd2d 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-topissues.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-topissues.inc @@ -25,7 +25,7 @@
  • [% END %] - [% AdvancedSearchTypes = Koha.Preference('AdvancedSearchTypes').split('\|') | html %] + [% AdvancedSearchTypes = Koha.Preference('AdvancedSearchTypes').split('\|') %] [% IF AdvancedSearchTypes.grep('^itemtypes$').size %]
  • diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt index c451d8d60d..f9a680de74 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -9,18 +9,18 @@ [% SET TagsShowEnabled = ( ( Koha.Preference( 'TagsEnabled' ) == 1 ) && TagsShowOnDetail ) %] [% SET TagsInputEnabled = ( ( Koha.Preference( 'opacuserlogin' ) == 1 ) && ( Koha.Preference( 'TagsEnabled' ) == 1 ) && TagsInputOnDetail ) %] [% IF Koha.Preference('AmazonAssocTag') %] - [% AmazonAssocTag = '?tag=' _ Koha.Preference('AmazonAssocTag') | html %] + [% AmazonAssocTag = '?tag=' _ Koha.Preference('AmazonAssocTag') %] [% ELSE %] - [% AmazonAssocTag = '' | html %] + [% AmazonAssocTag = '' %] [% END %] -[% ShowCourseReservesHeader = 0 | html %] +[% ShowCourseReservesHeader = 0 %] [% IF Koha.Preference( 'UseCourseReserves' ) == 1 %] [% FOREACH ITEM_RESULT IN itemloop %] [% IF ITEM_RESULT.course_reserves %] [% FOREACH r IN ITEM_RESULT.course_reserves %] [% IF r.course.enabled == 'yes' %] - [% ShowCourseReservesHeader = 1 | html %] + [% ShowCourseReservesHeader = 1 %] [% END %] [% END %] [% END %] @@ -78,13 +78,13 @@
    [% END %] [% IF ( Koha.Preference('Coce') && Koha.Preference('CoceProviders') ) %] - [% coce_id = normalized_ean || normalized_isbn | html %] + [% coce_id = normalized_ean || normalized_isbn %]
    [% END %] [% IF OpenLibraryCovers %]
    [% END %] - [% bt_id = ( normalized_upc || normalized_isbn ) | html %] + [% bt_id = ( normalized_upc || normalized_isbn ) %] [% IF ( BakerTaylorEnabled && bt_id ) %] [% IF BakerTaylorBookstoreURL %] [% IF ( OPACURLOpenInNewWindow ) %] @@ -168,7 +168,7 @@ [% END # /OPACPopupAuthorsSearch %] [% FOREACH MARCAUTHOR IN MARCAUTHORS %] - [% authorsloop = loop | html %] + [% authorsloop = loop %] [% FOREACH MARCAUTHOR_SUBFIELDS_LOO IN MARCAUTHOR.MARCAUTHOR_SUBFIELDS_LOOP %] [% MARCAUTHOR_SUBFIELDS_LOO.separator | html %] [% IF Koha.Preference( 'OPACPopupAuthorsSearch' ) == 1 %] @@ -268,7 +268,7 @@ All [% FOREACH MARCSUBJCT IN MARCSUBJCTS %] - [% subjectsloop = loop | html %] + [% subjectsloop = loop %] [% IF MARCSUBJCT.authoritylink %]