Bugzilla – Attachment 49690 Details for
Bug 14668
Show serial enumeration in the patron's opac checkout summary
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14668 - Show serial enumeration in the checkin page and the patron's opac checkout summary
Bug-14668---Show-serial-enumeration-in-the-checkin.patch (text/plain), 10.66 KB, created by
Charles Farmer
on 2016-03-30 16:12:02 UTC
(
hide
)
Description:
Bug 14668 - Show serial enumeration in the checkin page and the patron's opac checkout summary
Filename:
MIME Type:
Creator:
Charles Farmer
Created:
2016-03-30 16:12:02 UTC
Size:
10.66 KB
patch
obsolete
>From 43f60d481a1f90a0f5ec5b86347bf2d800d4ff57 Mon Sep 17 00:00:00 2001 >From: charles <charles.farmer@inlibro.com> >Date: Wed, 30 Mar 2016 12:10:29 -0400 >Subject: [PATCH] Bug 14668 - Show serial enumeration in the checkin page and > the patron's opac checkout summary > >The enumchron is now displayed alongside the author name in the "Title" section of the item summary. > >TEST PLAN >========= > > 1) INTRANET > > 1) Edit an item and add something in the "h" subfield, ie: "volume #42" > 2) Check out the item to a patron > 3) In the patron page, both at the bottom of the page or in the "Check out" tab, you should see the enumchron concatenated with the title > > 2) OPAC > > 1) Search for the same patron you used earlier > 2) In the "your summary" tab, you should see the enumchron in the Title > > 3) INTRANET > > 1) Check in the item you just borrowed > 2) At check in, the enumchron should also appear in the Title. >--- > C4/Reserves.pm | 1 + > circ/returns.pl | 1 + > koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js | 12 +++++- > .../prog/en/modules/circ/circulation.tt | 46 +++++++++++----------- > .../intranet-tmpl/prog/en/modules/circ/returns.tt | 2 +- > .../opac-tmpl/bootstrap/en/modules/opac-user.tt | 3 +- > svc/checkouts | 3 +- > 7 files changed, 41 insertions(+), 27 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index fbeca06..1731c92 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -368,6 +368,7 @@ sub GetReservesFromItemnumber { > $r->get_column('branchcode'), > $r->reserve_id(), > $r->waitingdate(), >+ $r->get_column('expirationdate'), > ); > } > >diff --git a/circ/returns.pl b/circ/returns.pl >index 2f09b80..27212e9 100755 >--- a/circ/returns.pl >+++ b/circ/returns.pl >@@ -594,6 +594,7 @@ foreach ( sort { $a <=> $b } keys %returneditems ) { > $ri{itemtitle} = $biblio->{'title'}; > $ri{itemauthor} = $biblio->{'author'}; > $ri{itemcallnumber} = $biblio->{'itemcallnumber'}; >+ $ri{enumchron} = $item->{'enumchron'}; > $ri{dateaccessioned} = $item->{dateaccessioned}; > $ri{itemtype} = $biblio->{'itemtype'}; > $ri{itemnote} = $biblio->{'itemnotes'}; >diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >index 5137b7d..15b36ac 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >+++ b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >@@ -227,7 +227,11 @@ $(document).ready(function() { > title += " " + value.subfield; > }); > >- title += "</a></span>"; >+ if ( oObj.enumchron ) { >+ title += "</a>" + " " + oObj.enumchron + "</span>"; >+ } else { >+ title += "</a></span>"; >+ } > > if ( oObj.author ) { > title += " " + BY.replace( "_AUTHOR_", " " + oObj.author ); >@@ -499,7 +503,11 @@ $(document).ready(function() { > title += " " + value.subfield; > }); > >- title += "</a></span>"; >+ if ( oObj.enumchron ) { >+ title += "</a>" + " " + oObj.enumchron + "</span>"; >+ } else { >+ title += "</a></span>"; >+ } > > if ( oObj.author ) { > title += " " + BY.replace( "_AUTHOR_", " " + oObj.author ); >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index 0efe49b..965ab3e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -800,28 +800,30 @@ No patron matched <span class="ex">[% message %]</span> > </ul> > </div> > >- [% IF WaitingHolds.count %] >- <div id="holdswaiting" class="circmessage"> >- <h4>Holds waiting:</h4> >- [% FOREACH w IN WaitingHolds %] >- <ul> >- <li> >- <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% w.biblio.biblionumber %]">[% w.biblio.title | html %]</a> >- ([% ItemTypes.GetDescription( w.item.effective_itemtype ) %]), >- [% IF ( w.biblio.author ) %] by [% w.biblio.author | html %] [% END %] >- [% IF ( w.item.itemcallnumber ) %] [[% w.item.itemcallnumber %]] [% END %] >- Hold placed on [% w.reservedate | $KohaDates %]. >- >- <br/> >- [% IF ( w.branch.branchcode == Branches.GetLoggedInBranchcode() ) %]<strong class="waitinghere">[% ELSE %]<strong>[% END %] >- [% SET waiting_expires_on = w.waiting_expires_on %] >- Waiting at [% w.branch.branchname | html %] [% IF waiting_expires_on %] until [% waiting_expires_on | $KohaDates %] [% END %] >- </strong> >- </li> >- </ul> >- [% END %] >- </div> >- [% END %] >+ [% IF ( WaitingReserveLoop ) %] >+ <div id="holdswaiting" class="circmessage"> >+ <h4>Holds waiting:</h4> >+ [% FOREACH WaitingReserveLoo IN WaitingReserveLoop %] >+ <ul> >+ <li> >+ <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% WaitingReserveLoo.biblionumber %]">[% WaitingReserveLoo.title | html %]</a> >+ ([% WaitingReserveLoo.itemtype %]) >+ [% IF ( WaitingReserveLoo.enumchron ) %] [% WaitingReserveLoo.enumchron %][% END %], >+ [% IF ( WaitingReserveLoo.author ) %]<br/>by [% WaitingReserveLoo.author | html %][% END %] >+ [% IF ( WaitingReserveLoo.itemcallnumber ) %][[% WaitingReserveLoo.itemcallnumber %]] [% END %] >+ Hold placed on [% WaitingReserveLoo.reservedate %]. >+ >+ <br /> >+ [% IF ( WaitingReserveLoo.waitingat ) %] >+ [% IF ( WaitingReserveLoo.waitinghere ) %]<strong class="waitinghere">[% ELSE %]<strong>[% END %] >+ Waiting at [% WaitingReserveLoo.waitingat | html %] [% IF WaitingReserveLoo.expiratedate %] until [% WaitingReserveLoo.expirationdate | $KohaDates %] [% END %] >+ </strong> >+ [% END %] >+ </li> >+ </ul> >+ [% END %] >+ </div> >+ [% END %]<!-- /If WaitingReserveLoop --> > > [% IF ( notes ) %] > <div id="circnotes" class="circmessage"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >index 2c650d9..df229ff 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >@@ -614,7 +614,7 @@ $(document).ready(function () { > [% END %] > </td> > <td class="ci-title"><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% riloo.itembiblionumber %]"> >- [% riloo.itemtitle |html %]</a></td> >+ [% riloo.itemtitle |html %]</a>[% IF ( riloo.enumchron ) %]<br/>[% riloo.enumchron %][% END %]</td> > <td class="ci-author">[% riloo.itemauthor %]</td> > <td class="ci-barcode"><a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% riloo.itembiblionumber %]&itemnumber=[% riloo.itemnumber %]#item[% riloo.itemnumber %]">[% riloo.barcode %]</a></td> > <td class="ci-homelibrary">[% Branches.GetName( riloo.homebranch ) %]</td> >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >index d45222e..5147701 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >@@ -193,6 +193,7 @@ > > <td class="title"> > <a class="title" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ISSUE.biblionumber %]">[% ISSUE.title |html %] [% FOREACH subtitl IN ISSUE.subtitle %] [% subtitl.subfield %][% END %]</a> >+ [% IF ( ISSUE.enumchron ) %] [% ISSUE.enumchron %][% END %] > <span class="item-details">[% ISSUE.author %]</span> > </td> > [% IF ( ISSUE.overdue ) %] >@@ -353,7 +354,7 @@ > <tr> > <td> > <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% i.item.biblio.biblionumber %]"> >- [% i.item.biblio.title %] >+ [% i.item.biblio.title %][% IF ( i.item.enumchron ) %] [% i.item.enumchron %][% END %] > </a> > </td> > >diff --git a/svc/checkouts b/svc/checkouts >index 93a1c95..3a3673f 100755 >--- a/svc/checkouts >+++ b/svc/checkouts >@@ -97,7 +97,7 @@ my $sql = ' > itemlost, > damaged, > location, >- >+ items.enumchron, > DATEDIFF( issuedate, CURRENT_DATE() ) AS not_issued_today > FROM issues > LEFT JOIN items USING ( itemnumber ) >@@ -173,6 +173,7 @@ while ( my $c = $sth->fetchrow_hashref() ) { > date_due_overdue => $c->{date_due_overdue} ? JSON::true : JSON::false, > timestamp => $c->{timestamp}, > onsite_checkout => $c->{onsite_checkout}, >+ enumchron => $c->{'enumchron'}, > renewals_count => $renewals_count, > renewals_allowed => $renewals_allowed, > renewals_remaining => $renewals_remaining, >-- >1.9.1
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 14668
:
45294
|
49690
|
49691
|
49711
|
53448
|
55756
|
55757
|
55758
|
55759
|
55806
|
55807
|
55808