Attachment 47524 Details for Bug 15697 – [SIGNED-OFF]Bug 15697: Unnecessary comma between title and subtitle on opac-detail.pl

[patch] [SIGNED-OFF]Bug 15697: Unnecessary comma between title and subtitle on opac-detail.pl

SIGNED-OFFBug-15697-Unnecessary-comma-between-titl.patch (text/plain), 2.60 KB, created by Héctor Eduardo Castro Avalos on 2016-02-01 17:13:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Héctor Eduardo Castro Avalos

Created: 2016-02-01 17:13:47 UTC

Size: 2.60 KB

patch

obsolete

>From 01b82587608b2ca17860743c831608c0e455b5a9 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 28 Jan 2016 20:33:58 +0000
>Subject: [PATCH] [SIGNED-OFF]Bug 15697: Unnecessary comma between title and
> subtitle on opac-detail.pl
>
>To test:
>1 - Go to Administration->Keyword to MARC mapping
>2 - Add a mapping (or ensure it exists)
>    Field name: subtitle
>    MARC field: 245
>    MARC subfield: b
>3 - View a record with a subtitle in the opac
>4 - Note in MARC 21 you have "Title of record:, subtitle of record"
>5 - Apply patch
>6 - View the record again and note the comma is no longer present.
>
>Note: this patch removes the comma only for MARC21 where subtitle is not
>repeatable. UNIMARC seems to be repeatable and does not include
>punctuation so I believe this may be needed there.
>
>Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
>Works as described. Comma removed from title of the window and breadcrumb
>section
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt |    4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>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 b9ab8cb..ba45596 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -24,7 +24,7 @@
>     [% SET protocol = "http://" %]
> [% END %]
> [% INCLUDE 'doc-head-open.inc' %]
>-<title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog &rsaquo; Details for: [% title %][% FOREACH subtitl IN subtitle %], [% subtitl.subfield %][% END %]</title>
>+<title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog &rsaquo; Details for: [% title %][% FOREACH subtitl IN subtitle %][% UNLESS Koha.Preference('marcflavour')=='MARC21' %],[% END %] [% subtitl.subfield %][% END %]</title>
> [% INCLUDE 'doc-head-close.inc' %]
> [% IF ( bidi ) %]
>   [% BLOCK cssinclude %]<link rel="stylesheet" type="text/css" href="[% interface %]/[% theme %]/css/right-to-left.css" />[% END %]
>@@ -40,7 +40,7 @@
> <div class="main">
>     <ul class="breadcrumb">
>         <li><a href="/cgi-bin/koha/opac-main.pl">Home</a> <span class="divider">&rsaquo;</span></li>
>-        <li><a href="#">Details for: [% title %][% FOREACH subtitl IN subtitle %], [% subtitl.subfield %][% END %]</a></li>
>+        <li><a href="#">Details for: [% title %][% FOREACH subtitl IN subtitle %][% UNLESS Koha.Preference('marcflavour')=='MARC21' %],[% END %] [% subtitl.subfield %][% END %]</a></li>
>     </ul>
> 
>     <div class="container-fluid">
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 15697: 47410 | 47517 | 47524 | 47554 | 47577 | 47578 | 48341 | 48342