Bugzilla – Attachment 188141 Details for
Bug 40818
marc_lib is mostly used raw in templates
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 40818: [22.11] Replace raw with html filter for marc_lib
Bug-40818-2211-Replace-raw-with-html-filter-for-ma.patch (text/plain), 19.98 KB, created by
Jonathan Druart
on 2025-10-20 07:49:18 UTC
(
hide
)
Description:
Bug 40818: [22.11] Replace raw with html filter for marc_lib
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2025-10-20 07:49:18 UTC
Size:
19.98 KB
patch
obsolete
>From 882c0ae50f6e007a3c491c8695bbd9efd5ce2e6a Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 20 Oct 2025 09:48:23 +0200 >Subject: [PATCH] Bug 40818: [22.11] Replace raw with html filter for marc_lib > >--- > .../prog/en/includes/html_helpers.inc | 4 ++-- > .../prog/en/modules/acqui/addorderiso2709.tt | 4 ++-- > .../en/modules/authorities/authorities.tt | 4 ++-- > .../prog/en/modules/cataloguing/addbiblio.tt | 8 ++++---- > .../prog/en/modules/serials/serials-edit.tt | 8 ++++---- > .../en/modules/services/itemrecorddisplay.tt | 4 ++-- > .../bootstrap/en/modules/opac-MARCdetail.tt | 20 +++++++++---------- > .../en/modules/opac-auth-MARCdetail.tt | 2 +- > 8 files changed, 27 insertions(+), 27 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc >index 9e2bb66190f..7360935809e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc >@@ -94,9 +94,9 @@ > <li> > <div class="subfield_line" style="[% subfield.visibility | html %]" id="subfield[% subfield.tag | html %][% subfield.subfield | html %][% subfield.random | html %]"> > [% IF ( subfield.mandatory ) %] >- <label class="required">[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label> >+ <label class="required">[% subfield.subfield | html %] - [% subfield.marc_lib | html %]</label> > [% ELSE %] >- <label>[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label> >+ <label>[% subfield.subfield | html %] - [% subfield.marc_lib | html %]</label> > [% END %] > > [% SET mv = subfield.marc_value %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >index 5b5907752b8..aaeea7c5cc3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >@@ -417,9 +417,9 @@ > <li style="[% iteminformatio.hidden | html %];"> > <div class="subfield_line" id="subfield[% iteminformatio.serialid | html %][% iteminformatio.countitems | html %][% iteminformatio.subfield | html %][% iteminformatio.random | html %]"> > [% IF (iteminformatio.mandatory) %] >- <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% ELSE %] >- <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% END %] > > [% IF ( iteminformatio.marc_value.type == 'select' ) %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >index a8af2418996..760b6b98ca0 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >@@ -493,7 +493,7 @@ > [% UNLESS hide_marc %] > <div class="subfieldcode"> > <input type="text" >- title="[% subfield_loo.marc_lib | $raw %]" >+ title="[% subfield_loo.marc_lib | html %]" > style=" [% IF ( subfield_loo.fixedfield ) %]display:none; [% END %]border:0;" > name="tag_[% subfield_loo.tag | html %]_code_[% subfield_loo.subfield | html %]_[% subfield_loo.index | html %]_[% subfield_loo.index_subfield | html %]" > value="[% subfield_loo.subfield | html %]" >@@ -519,7 +519,7 @@ > [% ELSE %] > <label for="tag_[% subfield_loo.tag | html %]_subfield_[% subfield_loo.subfield | html %]_[% subfield_loo.index | html %]_[% subfield_loo.index_subfield | html %]" class="labelsubfield"> > [% END %] >- [% subfield_loo.marc_lib | $raw %] >+ [% subfield_loo.marc_lib | html %] > </label> > </div> <!-- /.subfield --> > [% END # UNLESS advancedMARCEditor %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >index 55b3b992a65..5150e2e54b3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >@@ -565,7 +565,7 @@ $(document).ready(function(){ > [% FOREACH subfield_loo IN innerloo.subfield_loop %] > [% IF ( subfield_loo.mandatory ) %]subfields.push("[% subfield_loo.id | html %]"); > tab.push("[% BIG_LOO.number | html %]"); >- label.push("[% subfield_loo.marc_lib | $raw %]"); >+ label.push("[% subfield_loo.marc_lib | html %]"); > [% END %] > [% END %] > [% END %] >@@ -580,7 +580,7 @@ $(document).ready(function(){ > [% FOREACH subfield_loo IN innerloo.subfield_loop %] > [% IF ( subfield_loo.important ) %]subfields.push("[% subfield_loo.id | html %]"); > tab.push("[% BIG_LOO.number | html %]"); >- label.push("[% subfield_loo.marc_lib | $raw %]"); >+ label.push("[% subfield_loo.marc_lib | html %]"); > [% END %] > [% END %] > [% END %] >@@ -1110,7 +1110,7 @@ $(document).ready(function(){ > <li class="subfield_line" style="[% subfield_loo.visibility | html %]" id="subfield[% subfield_loo.tag | html %][% subfield_loo.subfield | html %][% subfield_loo.random | html %]"> > <div class="subfieldcode"> > <input type="text" >- title="[% subfield_loo.marc_lib | $raw %]" >+ title="[% subfield_loo.marc_lib | html %]" > style=" [% IF ( subfield_loo.fixedfield ) %]display:none; [% END %]border:0;" > name="tag_[% subfield_loo.tag | html %]_code_[% subfield_loo.subfield | html %]_[% subfield_loo.index | html %]_[% subfield_loo.index_subfield | html %]" > value="[% subfield_loo.subfield | html %]" >@@ -1132,7 +1132,7 @@ $(document).ready(function(){ > [% ELSE %] > <label for="tag_[% subfield_loo.tag | html %]_subfield_[% subfield_loo.subfield | html %]_[% subfield_loo.index | html %]_[% subfield_loo.index_subfield | html %]" class="labelsubfield"> > [% END %] >- [% subfield_loo.marc_lib | $raw %] >+ [% subfield_loo.marc_lib | html %] > </label> > </div> > [% END %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt >index d7fe47b7c42..85eada3667c 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt >@@ -252,9 +252,9 @@ $(document).ready(function() { > <ol>[% FOREACH iteminformatio IN item.iteminformation %]<li> > <div class="subfield_line" style="[% iteminformatio.hidden | html %]" id="subfield[% item.serialid | html %][% item.countitems | html %][% iteminformatio.subfield | html %][% iteminformatio.random | html %]"> > [% IF (iteminformatio.mandatory) %] >- <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% ELSE %] >- <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% END %] > [% IF ( iteminformatio.marc_value.type == 'select' ) %] > <select name="field_value" class="select2 input_marceditor"> >@@ -404,9 +404,9 @@ $(document).ready(function() { > <ol>[% FOREACH iteminformatio IN newserialloo.iteminformation %]<li> > <div class="subfield_line" style="[% iteminformatio.hidden | html %]" id="subfield[% newserialloo.serialid | html %][% newserialloo.countitems | html %][% iteminformatio.subfield | html %][% iteminformatio.random | html %]"> > [% IF (iteminformatio.mandatory) %] >- <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label class="required">[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% ELSE %] >- <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | $raw %]</label> >+ <label>[% iteminformatio.subfield | html %] - [% iteminformatio.marc_lib | html %]</label> > [% END %] > [% IF ( iteminformatio.marc_value.type == 'select' ) %] > <select name="field_value" class="select2 input_marceditor"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt >index c8a3b1fd24d..04a73eb460c 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt >@@ -8,9 +8,9 @@ > [% END %] > <div class="subfield_line" id="subfield[% iteminfo.serialid | html %][% iteminfo.countitems | html %][% iteminfo.subfield | html %][% iteminfo.random | html %]"> > [% IF (iteminfo.mandatory) %] >- <label class="required">[% iteminfo.subfield | html %] - [% iteminfo.marc_lib | $raw %]</label> >+ <label class="required">[% iteminfo.subfield | html %] - [% iteminfo.marc_lib | html %]</label> > [% ELSE %] >- <label>[% iteminfo.subfield | html %] - [% iteminfo.marc_lib | $raw %]</label> >+ <label>[% iteminfo.subfield | html %] - [% iteminfo.marc_lib | html %]</label> > [% END %] > [% IF ( iteminfo.marc_value.type == 'select' ) %] > <select name="field_value" class="input_marceditor"> >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt >index 392ca0b3acf..f5e78adcc79 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt >@@ -43,7 +43,7 @@ > <tr><th colspan="2">[% tab0X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab0X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -52,7 +52,7 @@ > <tr><th colspan="2">[% tab1X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab1X.subfield %] > <tr> >- <td >[% subfiel.marc_lib | $raw %]</td> >+ <td >[% subfiel.marc_lib | html %]</td> > <td >[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -62,7 +62,7 @@ > <tr><th colspan="2">[% tab2X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab2X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -72,7 +72,7 @@ > <tr><th colspan="2">[% tab3X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab3X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -82,7 +82,7 @@ > <tr><th colspan="2" >[% tab4X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab4X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -92,7 +92,7 @@ > <tr><th colspan="2">[% tab5X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab5X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -102,7 +102,7 @@ > <tr><th colspan="2">[% tab6X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab6X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -112,7 +112,7 @@ > <tr><th colspan="2">[% tab7X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab7X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -122,7 +122,7 @@ > <tr><th colspan="2">[% tab8X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab8X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >@@ -132,7 +132,7 @@ > <tr><th colspan="2">[% tab9X.tag | html %]</th></tr> > [% FOREACH subfiel IN tab9X.subfield %] > <tr> >- <td>[% subfiel.marc_lib | $raw %]</td> >+ <td>[% subfiel.marc_lib | html %]</td> > <td>[% subfiel.marc_value | html %]</td> > </tr> > [% END %] >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-MARCdetail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-MARCdetail.tt >index ecec5b0de60..321c4a3cae8 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-MARCdetail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-MARCdetail.tt >@@ -66,7 +66,7 @@ > <p><strong>[% Tag0X.tag | html %]</strong></p> > <ul> > [% FOREACH subfiel IN Tag0X.subfield %] >- <li><strong>[% subfiel.marc_lib | $raw %]:</strong> >+ <li><strong>[% subfiel.marc_lib | html %]:</strong> > [% IF subfiel.isurl %]<a href="[% subfiel.marc_value | url %]">[% subfiel.marc_value | html %]</a> > [% ELSE %][% subfiel.marc_value | html %][% END %]</li> > [% END %] >-- >2.34.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 40818
:
187503
|
187504
|
187782
|
187783
|
187784
|
188053
|
188054
|
188055
|
188140
| 188141 |
188531
|
188532