Bugzilla – Attachment 20053 Details for
Bug 10672
Add subtitle to display of checkouts, overdues, and holds on the patron summary
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 10672 - Add subtitle to display of checkouts, overdues, and holds on the patron summary
Bug-10672---Add-subtitle-to-display-of-checkouts-o.patch (text/plain), 4.14 KB, created by
Srdjan Jankovic
on 2013-08-02 02:48:15 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 10672 - Add subtitle to display of checkouts, overdues, and holds on the patron summary
Filename:
MIME Type:
Creator:
Srdjan Jankovic
Created:
2013-08-02 02:48:15 UTC
Size:
4.14 KB
patch
obsolete
>From 4b2043f84f09e6d3ecf15a14d564585a9c743ad6 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >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 > >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. > >Signed-off-by: Srdjan <srdjan@catalyst.net.nz> >--- > 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({ > > </td>[% END %] > >- <td class="title"><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ISSUE.biblionumber %]">[% ISSUE.title |html %]</a><span class="item-details"> >+ <td class="title"><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ISSUE.biblionumber %]">[% ISSUE.title |html %][% FOREACH subtitl IN ISSUE.subtitle %] [% subtitl.subfield %][% END %]</a><span class="item-details"> > [% ISSUE.author %] > </span></td> > [% IF ( ISSUE.overdue ) %] >@@ -325,7 +325,7 @@ $.tablesorter.addParser({ > [% END %] > </td>[% END %] > >-<td><a href="/cgi-bin/koha/opac-detail.pl?bib=[% OVERDUE.biblionumber %]">[% OVERDUE.title |html %]</a> <span class="item-details">[% OVERDUE.author %]</span></td> >+<td><a href="/cgi-bin/koha/opac-detail.pl?bib=[% OVERDUE.biblionumber %]">[% OVERDUE.title |html %][% FOREACH subtitl IN OVERDUE.subtitle %] [% subtitl.subfield %][% END %]</a> <span class="item-details">[% OVERDUE.author %]</span></td> > > [% UNLESS ( item_level_itypes ) %]<td>[% IF ( OVERDUE.imageurl ) %]<img src="[% OVERDUE.imageurl %]" title="[% OVERDUE.description %]" alt="[% OVERDUE.description %]" />[% END %] [% OVERDUE.description %]</td>[% END %] > [% IF ( show_barcode ) %]<td>[% OVERDUE.barcode %]</td>[% END %] >@@ -381,7 +381,7 @@ $.tablesorter.addParser({ > [% ELSE %] > <tr> > [% END %] >- <td class="title"><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% RESERVE.biblionumber %]">[% RESERVE.reserves_title %]</a> >+ <td class="title"><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% RESERVE.biblionumber %]">[% RESERVE.reserves_title %][% FOREACH subtitl IN RESERVE.subtitle %] [% subtitl.subfield %][% END %]</a> > [% RESERVE.author %] > </td> > <td class="reservedate">[% RESERVE.reservedate | $KohaDates %]</td> >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.8.1.2
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 10672
:
20043
|
20053
|
20836