Bugzilla – Attachment 20759 Details for
Bug 10221
hold expiration doesn't show on patron's list of holds
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[3.10.x] Bug 10221 - hold expiration doesn't show on patron's list of holds
Bug-10221---hold-expiration-doesnt-show-on-patrons.patch (text/plain), 4.49 KB, created by
Jonathan Druart
on 2013-09-02 07:24:19 UTC
(
hide
)
Description:
[3.10.x] Bug 10221 - hold expiration doesn't show on patron's list of holds
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-09-02 07:24:19 UTC
Size:
4.49 KB
patch
obsolete
>From e10eaf8bc1b434f3e31a6937fad8e975d198c206 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Mon, 13 May 2013 14:26:17 -0400 >Subject: [PATCH] Bug 10221 - hold expiration doesn't show on patron's list of > holds > >When viewing the list of a patron's holds on the circulation page or the >patron detail page the table of holds doesn't display the holds >expiration date, if present. This patch adds display of the hold >expiration date. > >To test, place a hold for a patron and set an expiration date for it. >View the patron's existing holds on both circulation.pl and >moremember.pl. The expiration date you chose should appear in the table >of information about existing holds. >--- > circ/circulation.pl | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 2 ++ > koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt | 4 +++- > members/moremember.pl | 1 + > 4 files changed, 7 insertions(+), 1 deletion(-) > >diff --git a/circ/circulation.pl b/circ/circulation.pl >index 0dfd4f7..cc674b7 100755 >--- a/circ/circulation.pl >+++ b/circ/circulation.pl >@@ -407,6 +407,7 @@ if ($borrowernumber) { > $getreserv{biblionumber} = $num_res->{'biblionumber'}; > } > $getreserv{waitingposition} = $num_res->{'priority'}; >+ $getreserv{expirationdate} = $num_res->{'expirationdate'}; > push( @reservloop, \%getreserv ); > > # if we have a reserve waiting, initiate waitingreserveloop >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 3b3bbb0..c0a7b2b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -1066,6 +1066,7 @@ No patron matched <span class="ex">[% message %]</span> > <th>Title</th> > <th>Call number</th> > <th>Barcode</th> >+ <th>Expiration</th> > <th>Priority</th> > <th>Delete?</th> > <th> </th> >@@ -1084,6 +1085,7 @@ No patron matched <span class="ex">[% message %]</span> > [% END %] > [% IF ( reservloo.nottransfered ) %] hasn't been transferred yet from [% reservloo.nottransferedby %]</i> > [% END %]</em></td> >+ <td>[% reservloo.expirationdate | $KohaDates %]</td> > <td> > [% IF ( reservloo.waitingposition ) %]<b> [% reservloo.waitingposition %] </b>[% END %] > </td> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >index 39f3e55..38c37aa 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >@@ -97,7 +97,7 @@ if (nodename =="barcodes[]"){ > var Node=me.parentNode.previousSibling; > while (Node.nodeName!="TD"){Node=Node.previousSibling} > var Nodes=Node.childNodes; >- for (var i=0;i<Nodes.length;i++){ >+ for (var i=0;i < Nodes.length;i++){ > if (Nodes[i].nodeName=="INPUT" && Nodes[i].getAttribute("type")=="checkbox"){ > Nodes[i].checked=false; > } >@@ -603,6 +603,7 @@ function validate1(date) { > <th>Title</th> > <th>Call number</th> > <th>Barcode</th> >+ <th>Expiration</th> > <th>Priority</th> > <th>Delete?</th> > <th> </th> >@@ -629,6 +630,7 @@ function validate1(date) { > <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% reservloo.biblionumber %]">[% reservloo.barcodereserv %]</a> > [% END %] > </td> >+ <td>[% reservloo.expirationdate | $KohaDates %]</td> > <td>[% IF ( reservloo.waitingposition ) %]<strong>[% reservloo.waitingposition %]</strong>[% END %]</td> > <td><select name="rank-request"> > <option value="n">No</option> >diff --git a/members/moremember.pl b/members/moremember.pl >index 0524ecb..4acd2eb 100755 >--- a/members/moremember.pl >+++ b/members/moremember.pl >@@ -322,6 +322,7 @@ if ($borrowernumber) { > $getreserv{waitingposition} = $num_res->{'priority'}; > $getreserv{suspend} = $num_res->{'suspend'}; > $getreserv{suspend_until} = $num_res->{'suspend_until'}; >+ $getreserv{expirationdate} = $num_res->{'expirationdate'}; > > push( @reservloop, \%getreserv ); > } >-- >1.7.10.4
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 10221
:
18118
|
18120
|
18742
| 20759