Bug 10221 - hold expiration doesn't show on patron's list of holds
Summary: hold expiration doesn't show on patron's list of holds
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: 3.14
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Jonathan Druart
URL: /cgi-bin/koha/circ/circulation.pl?bor...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-08 19:18 UTC by Nicole C. Engard
Modified: 2014-12-07 20:03 UTC (History)
4 users (show)

See Also:
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 (4.85 KB, patch)
2013-05-13 18:33 UTC, Owen Leonard
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 10221 - hold expiration doesn't show on patron's list of holds (4.89 KB, patch)
2013-05-13 19:28 UTC, Nicole C. Engard
Details | Diff | Splinter Review
Bug 10221 - hold expiration doesn't show on patron's list of holds (4.99 KB, patch)
2013-06-07 12:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
[3.10.x] Bug 10221 - hold expiration doesn't show on patron's list of holds (4.49 KB, patch)
2013-09-02 07:24 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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.