Bug 10221

Summary: hold expiration doesn't show on patron's list of holds
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, jonathan.druart, sophie.meynieux, tomascohen
Version: 3.14   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/circ/circulation.pl?borrowernumber=51
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10221 - hold expiration doesn't show on patron's list of holds
[SIGNED-OFF] Bug 10221 - hold expiration doesn't show on patron's list of holds
Bug 10221 - hold expiration doesn't show on patron's list of holds
[3.10.x] Bug 10221 - hold expiration doesn't show on patron's list of holds

Description Nicole C. Engard 2013-05-08 19:18:58 UTC
When looking at the patron details in the staff client you don't see the holds expiration date, but you do see all the other dates (placed, suspension) We should see the same here:

/cgi-bin/koha/circ/circulation.pl?borrowernumber=51

that we see here:

/cgi-bin/koha/reserve/request.pl?biblionumber=3174

When it relates to hold dates.
Comment 1 Owen Leonard 2013-05-13 18:33:41 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2013-05-13 19:28:21 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-06-07 12:57:54 UTC
QA comment:
The patch adds a "expiration" column for holds tables on moremember and circulation pages.
Works as expected.
Marked as Passed QA.
Comment 4 Jonathan Druart 2013-06-07 12:58:10 UTC Comment hidden (obsolete)
Comment 5 Galen Charlton 2013-06-07 15:52:07 UTC
Pushed to master.  Thanks, Owen!
Comment 6 Tomás Cohen Arazi 2013-06-11 18:39:24 UTC
This patch has been pushed to 3.12.x, will be in 3.12.1.
I consider this an usability bugfix.

Thanks Owen!
Comment 7 Sophie MEYNIEUX 2013-08-30 16:20:18 UTC
Would it be possible to have it in 3.10 too ? One of our customer would be delighted :-)
Comment 8 Jonathan Druart 2013-09-02 07:24:19 UTC
Created attachment 20759 [details] [review]
[3.10.x] 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.