Attachment 126996 Details for Bug 29075 – Bug 29075: 'OPAC info' removed in branches.pl

[patch] Bug 29075: 'OPAC info' removed in branches.pl

Bug-29075-OPAC-info-removed-in-branchespl.patch (text/plain), 1.94 KB, created by Jérémy Breuillard on 2021-10-27 13:30:09 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jérémy Breuillard

Created: 2021-10-27 13:30:09 UTC

Size: 1.94 KB

patch

obsolete

>From c86509ed1d707497a73a83bcd97c5bc806bdca61 Mon Sep 17 00:00:00 2001
>From: jeremy breuillard <jeremy.breuillard@biblibre.com>
>Date: Wed, 27 Oct 2021 13:24:43 +0000
>Subject: [PATCH] Bug 29075: 'OPAC info' removed in branches.pl
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>In Administration âº Libraries, we see content of OPAC info as escaped HTML.
>This content may be long and seeing HTML tags is strange.
>
>We should not show it in this table.
>Or maybe create a modal preview of it (not escaped HTML).
>
>To test :
>1) Home > Administration > Libraries
>2) In 'Address' column notice the 'OPAC info' field (if this one is
>filled) with visible HTML tags
>3) Apply patch
>4) Repeat 1) and 'OPAC info' field should be gone
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>index 6ce5842..22e4c8f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>@@ -385,7 +385,6 @@ Libraries &rsaquo; Administration &rsaquo; Koha
>                             if ( row.fax != null ) library_info.push(_("Fax: ") + row.fax.escapeHtml());
>                             if ( row.email != null ) library_info.push('<a href="mailto:'+encodeURIComponent(row.email)+'">'+row.email.escapeHtml()+'</a>');
>                             if ( row.url != null ) library_info.push('<a href="'+encodeURIComponent(row.url)+'">'+row.url.escapeHtml()+'</a>');
>-                            if ( row.opac_info != null ) library_info.push(_("OPAC Info") + ': <div>'+row.opac_info.escapeHtml()+'</div>');
>                             if ( row.notes != null ) library_info.push(_("Notes")+': '+row.notes.escapeHtml());
>                             return library_info.join('<br/>');
>                         },
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 29075: 126975 | 126987 | 126994 | 126996 | 127072 | 127127