TPAC reports hold "ready for pickup" before hold_shelf_status_delay expires

Bug #1195428 reported by Bill Ott
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Medium
Unassigned

Bug Description

ver. 2.2, but suspected all versions

In the summary of myopac, the integer for "Items ready for pickup" will display a count for items that are meant to be masked from the patron via circ.hold_shelf_status_delay. Confusion is caused when the individual hold display does not report the same number of holds actually available for pickup.

In Actor.pm, open-ils.actor.user.hold_requests.count, does not take into consideration the org_unit_setting noted.

Revision history for this message
Bill Ott (bott) wrote :

I've created a patch which will consider the aous for each current_shelf_lib and compare the delay before displaying the count value.

Unfortunately, the OPAC and the staff client both utilize this same code. So, both will impacted by the delay.

Revision history for this message
Bill Ott (bott) wrote :

Argh! I missed a line in the commit.

Revision history for this message
Ben Shum (bshum) wrote :

Hey Bill, can you throw this into a branch with your proper name/git signoff? The patches included are from one of your servers I'm guessing.

Thanks!

Changed in evergreen:
status: New → Incomplete
importance: Undecided → Medium
tags: added: holds opac tpac
Revision history for this message
Shae (shae-esilibrary) wrote :

At version 2.4.2 no patch appears to be in place for this. I tested on a training system in house to confirm it is still happening. Hold shelf status delay is set to 30 minutes but, as soon as I captured two holds for my account, it shows in the staff client and TPAC 2 holds ready for pickup for the patron. As far as I know from working with customers, this delay is working as expected as far as preventing the user from getting an actual email notification of the holds being ready for pickup.

Revision history for this message
Erica Rohlfs (erohlfs) wrote :

I marked this bug invalid, because the library setting does control the behavior of when we see the Status of the lineitem on hold update. Until the time interval passes as defined by the Library Setting: Hold Shelf Status Delay, the hold's Status column continues to display the Estimated Wait (on an out-of-the-box configuration). Once the time interval has passed, the Status column updates with Available for the hold.

Revision history for this message
Kathy Lussier (klussier) wrote :

I'm going to bounce this one back to Confirmed and re-target to supported releases. This is what I see when using the setting on the stock system. I set the delay to one hour. When the hold is captured, I log into the patron's account:

The status column does indeed show x hold on y copies in the status column when the patron accesses my account. This is what Erica describes, but is not the problem area identified in the original report.

In the patron's dashboard, it displays as "1 Ready for Pickup". This is the bug. It shouldn't be showing that there are any holds ready for pickup.

If I click on the "1 Ready for Pickup" link or if I click the link to only show available holds, it will show the hold that was just captured, even though the status doesn't yet identify it as being available.

Changed in evergreen:
status: Incomplete → Confirmed
tags: added: needsrepatch
Revision history for this message
Kathy Lussier (klussier) wrote :

Actually, I decided not to re-target to current release since it looks like work is still required on the patch. But I set 2.3 and 2.4 to Won't Fix.

tags: removed: tpac
no longer affects: evergreen/2.3
no longer affects: evergreen/2.4
tags: added: circ-holds needswork
removed: holds needsrepatch
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.