From 81aa0a7318d125b2c24042a0b6efe023986aeb1f Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 1 Aug 2013 10:57:02 -0400 Subject: [PATCH] Bug 10672 - Add subtitle to display of checkouts, overdues, and holds on the patron summary Content-Type: text/plain; charset="utf-8" In the OPAC, the patron summary page (opac-user.pl) should display subtitle along with title in the lists of checkouts, overdues, and holds. This patch adds it. To test, log in to the OPAC as a patron who has checkouts, overdues, and holds which include titles with subtitles. Titles should appear correctly with and without subtitles. --- koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt | 6 +++--- opac/opac-user.pl | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt index d7c691f..f75c82c 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt @@ -179,7 +179,7 @@ $.tablesorter.addParser({ [% END %] - [% ISSUE.title |html %] + [% ISSUE.title |html %][% FOREACH subtitl IN ISSUE.subtitle %] [% subtitl.subfield %][% END %] [% ISSUE.author %] [% IF ( ISSUE.overdue ) %] @@ -325,7 +325,7 @@ $.tablesorter.addParser({ [% END %] [% END %] -[% OVERDUE.title |html %] [% OVERDUE.author %] +[% OVERDUE.title |html %][% FOREACH subtitl IN OVERDUE.subtitle %] [% subtitl.subfield %][% END %] [% OVERDUE.author %] [% UNLESS ( item_level_itypes ) %][% IF ( OVERDUE.imageurl ) %][% OVERDUE.description %][% END %] [% OVERDUE.description %][% END %] [% IF ( show_barcode ) %][% OVERDUE.barcode %][% END %] @@ -381,7 +381,7 @@ $.tablesorter.addParser({ [% ELSE %] [% END %] - [% RESERVE.reserves_title %] + [% RESERVE.reserves_title %][% FOREACH subtitl IN RESERVE.subtitle %] [% subtitl.subfield %][% END %] [% RESERVE.author %] [% RESERVE.reservedate | $KohaDates %] diff --git a/opac/opac-user.pl b/opac/opac-user.pl index 5c01ae2..731caf8 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -176,7 +176,7 @@ if ($issues){ } } $issue->{'charges'} = $charges; - + $issue->{'subtitle'} = GetRecordValue('subtitle', GetMarcBiblio($issue->{'biblionumber'}), GetFrameworkCode($issue->{'biblionumber'})); # check if item is renewable my ($status,$renewerror) = CanBookBeRenewed( $borrowernumber, $issue->{'itemnumber'} ); ($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'}); @@ -256,7 +256,7 @@ foreach my $res (@reserves) { if ( $res->{'expirationdate'} eq '0000-00-00' ) { $res->{'expirationdate'} = ''; } - + $res->{'subtitle'} = GetRecordValue('subtitle', GetMarcBiblio($res->{'biblionumber'}), GetFrameworkCode($res->{'biblionumber'})); $res->{'waiting'} = 1 if $res->{'found'} eq 'W'; $res->{'branch'} = $branches->{ $res->{'branchcode'} }->{'branchname'}; my $biblioData = GetBiblioData($res->{'biblionumber'}); -- 1.7.9.5