Bugzilla – Attachment 187783 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: Replace raw with html filter for marc_lib
Bug-40818-Replace-raw-with-html-filter-for-marclib.patch (text/plain), 20.21 KB, created by
David Cook
on 2025-10-13 02:43:59 UTC
(
hide
)
Description:
Bug 40818: Replace raw with html filter for marc_lib
Filename:
MIME Type:
Creator:
David Cook
Created:
2025-10-13 02:43:59 UTC
Size:
20.21 KB
patch
obsolete
>From 60d4344e824d4bda7e6d9b32dd18df9f68c83459 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Tue, 7 Oct 2025 09:32:31 +0200 >Subject: [PATCH] Bug 40818: Replace raw with html filter for marc_lib > >Signed-off-by: David Cook <dcook@prosentient.com.au> >--- > .../prog/en/modules/acqui/addorderiso2709.tt | 4 ++-- > .../en/modules/authorities/authorities.tt | 6 +++--- > .../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 +- > 7 files changed, 26 insertions(+), 26 deletions(-) > >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 d0e834b57a..e45dff7e18 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >@@ -531,9 +531,9 @@ > 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 d9e91e55c0..bc0b357e48 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >@@ -218,7 +218,7 @@ > [% 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 %] >@@ -680,7 +680,7 @@ > <div class="subfieldcode"> > <span class="handle"></span> > <input type="text" >- title="[% subfield_loo.marc_lib | $raw %]" >+ title="[% subfield_loo.marc_lib | html %]" > style=" [% IF ( innerloo.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 %]" >@@ -706,7 +706,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 7d75f7970c..7a4b5ba024 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >@@ -537,7 +537,7 @@ > [% 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 %] >@@ -552,7 +552,7 @@ > [% 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 %] >@@ -1131,7 +1131,7 @@ > <div class="subfieldcode"> > <span class="handle"></span> > <input type="text" >- title="[% subfield_loo.marc_lib | $raw %]" >+ title="[% subfield_loo.marc_lib | html %]" > style=" [% IF ( innerloo.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 %]" >@@ -1153,7 +1153,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> > [% 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 40d5f4cfaf..b7e742623b 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 >@@ -342,9 +342,9 @@ > 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"> >@@ -538,9 +538,9 @@ > 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 412ba08f89..aeffeac536 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt >@@ -4,9 +4,9 @@ > <li style="[% iteminfo.hidden | html %];"> > <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 2279b6606b..20d8cb99be 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-MARCdetail.tt >@@ -53,7 +53,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 %] >@@ -62,7 +62,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 %] >@@ -72,7 +72,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 %] >@@ -82,7 +82,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 %] >@@ -92,7 +92,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 %] >@@ -102,7 +102,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 %] >@@ -112,7 +112,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 %] >@@ -122,7 +122,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 %] >@@ -132,7 +132,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 %] >@@ -142,7 +142,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 7fe6d394a6..c56f3cc8e0 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 >@@ -72,7 +72,7 @@ > <ul> > [% FOREACH subfiel IN Tag0X.subfield %] > <li >- ><strong>[% subfiel.marc_lib | $raw %]:</strong> [% IF subfiel.isurl %] >+ ><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 %] >-- >2.39.5
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