Summary: | If an item is reserved, at the opac it is marked as available | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED WONTFIX | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | minor | ||
Priority: | P5 - low | CC: | fridolin.somers, jdemuth, katrin.fischer, m.de.rooy, mirko, paul.poulain |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3333 http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7012 |
||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 9253: Mark items "On hold" at the opac
Bug 9253: Mark items "On hold" at the opac |
Description
Jonathan Druart
2012-12-10 14:25:49 UTC
Created attachment 13978 [details] [review] Bug 9253: Mark items "On hold" at the opac If an item is reserved, at the opac it is marked as available. Test plan: - Reserve an item for a patron - See the intranet display page for the reserved notice. The item is marked as "on hold" - See the opac display page for the reserved notice. The item is marked as "available" - apply the patch - See the opac diplay page for the reserved notice. The item is marked as "on hold" Created attachment 14105 [details] [review] Bug 9253: Mark items "On hold" at the opac If an item is reserved, at the opac it is marked as available. Test plan: - Reserve an item for a patron - See the intranet display page for the reserved notice. The item is marked as "on hold" - See the opac display page for the reserved notice. The item is marked as "available" - apply the patch - See the opac diplay page for the reserved notice. The item is marked as "on hold" Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> Patch has described effect on OPAC. Opposed to the test plan I have 'available' in intranet though. Not in the scope of this patch so signed off. I am a bit worried about the consequences of this. I think this can only work for item level holds. Also, when does the status change? When the item is placed on hold or when the item is checked in and waiting for the patron? I feel like it should be the latter. I think libraries allow patrons to check out the item as long as it has not been pulled from the shelf, but not quite sure here. (In reply to comment #3) > I am a bit worried about the consequences of this. I think this can only > work for item level holds. You're right and wrong. This work only when the *item* is on hold. So either : - a "1st available" has been placed, and an item has been checked in (to the reserve is now attached to an item) - a item level hold has been placed > Also, when does the status change? The onhold flag is set by if( $reserve_status eq "Reserved"){ $itm->{'onhold'} = 1; } $reserve_status is defined by my ($reserve_status) = C4::Reserves::CheckReserves($itm->{itemnumber}); However, one thing that worry me is that the rules for opac-detail.pl display are not the same as the rules for staff detail.pl display: In staff display, it seems that the items.reservedate is used to have a display of hold-er & hold status. Jonathan, am I right ? Why use a different method in the staff interface ? > When the item is placed on hold or when the item is checked in and waiting > for the patron? I feel like it should be the latter. > I think libraries allow patrons to check out the item as long as it has not > been pulled from the shelf, but not quite sure here. I think we must : * be consistent between staff & opac * can display that an item is reserved as soon as possible. use case = 9AM, patron A places a hold, 10AM, patron B check OPAC and see the document is "available", 11AM, the item become "waiting" for patron A, 11:10AM, patron B arrives at the library and becomes angry because the item is no more available... We can't warn an item is reserved when the hold is "1st available", but as soon as we've an item, we should. (In reply to comment #4) > Jonathan, am I right ? Why use a different method in the staff interface ? I don't know. The only thing I know is that the opac template does not take into account the on hold status. QA: Looking at this one now.. QA Comment: I am sorry, but IMO this should not be pushed in this state. There already is a small discrepancy between opac and staff in this area. If we change things at one side now, let us resolve these issues and not make it bigger. IMO this is in scope. A next available hold is shown as Available in opac and staff until it is confirmed. I think that is fine. Does not need a change as proposed here; there may be a hold already for a far future! It is available until we check it in. After confirming, it is On Hold or Waiting.. in staff. Fine too. An item level hold is now Available in opac and marked as item-level hold in staff while pending. Small discrepancy. Note that this item level hold can also be still far away! Although I can understand your wish to put it on hold in display, I still think that it is actually available until we check it in. After confirming, there is no change anymore with the next available hold. So, if we want to change this status, the least thing we can do is first look at the reserve date before saying that it is not available. Setting to Failed QA I think Marcel managed to describe the problem quite well. I still feel like for the OPAC user a hold does not exist, until he can't check out the book or the book has been pulled and is no longer on the shelf. For staff it might make sense to give a bit more information. Not sure why this bug was assigned to me, I am not working on this so I reset it to default |