Bug 4218

Summary: Staff client detail page does not show item hold status
Product: Koha Reporter: Owen Leonard <oleonard>
Component: SearchingAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, koha.sekjal, nengard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Screenshot of proposed page display
Proposed fix
Follow-up fix for XSLT display

Description Chris Cormack 2010-05-21 01:24:32 UTC


---- Reported by oleonard@myacpl.org 2010-02-18 19:13:06 ----

In the staff client, the detail page does not show whether an item is on hold. An on-hold item will say "available" in the status column.



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:24 UTC  ---

This bug was previously known as _bug_ 4218 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4218

Actual time not defined. Setting to 0.0
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.

Comment 1 Ian Walls 2010-07-13 18:07:50 UTC
This can be remedied for any item-level holds, but what about title-level holds?  If the biblio in question has 3 copies attached, all checked in, which should be listed as 'on hold' for the patron?

The hold_fill_targets, part of the holds/build_holds_queue.pl cronjob, is the only place in the code so far that seems to do the job of assigning a title-level hold to a specific copy based on the circulation configurations, and it is only effected by the (typically hourly) cron.
Comment 2 Owen Leonard 2010-09-21 16:42:20 UTC
Created attachment 2638 [details]
Screenshot of proposed page display

I suggest adding the hold count as part of the biblio-level information being displayed.
Comment 3 Owen Leonard 2010-09-21 16:54:59 UTC
Created attachment 2639 [details] [review]
Proposed fix

Showing "on hold" in the holdings list (as we used to do) is misleading
if it is a bibilo-level hold. However seeing that a title has holds
is important information for this view. This patch adds a line to the
biblio-level information display with a count of the title's outstanding
holds.
Comment 4 Nicole C. Engard 2010-09-21 18:41:35 UTC
Love it - Signed off.
Comment 5 Chris Cormack 2010-12-13 02:58:04 UTC
Pushed to master, please test
Comment 6 Owen Leonard 2011-03-16 15:37:49 UTC
Created attachment 3325 [details] [review]
Follow-up fix for XSLT display
Comment 7 Chris Cormack 2011-03-17 21:53:27 UTC
Pushed, please test