Need to allow for multiple title holds

Bug #1654369 reported by Jason Boyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
NCIPServer
Confirmed
Wishlist
Unassigned

Bug Description

Given this situation:
Lib A has 2 copies of title 1
Lib B has 2 patrons, x and y that each want to request title 1
Lib A is running Evergreen and NCIPServer with a patron account B for Lib B transit purposes

When done in this order:
x places request with resource sharing system
rss sends RequestItem message to Lib A, hold is placed for B
y places request with rss
rss sends RequestItem message to Lib A, hold fails because B already has a hold on title 1 for patron x

y (or any other Lib B patron) can't request this item until after x has already checked it out, at which time they can orderly queue up one at a time as frustration mounts.

My Q&D hack to get this to work was to simply always call the .override version of holds.test_and_create.batch and give the ncip user account teh HOLDS_EXIST.override permission. I'm not certain if this would be preferred over calling the non-override version first and then using the override for this specific failure.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Do you have a branch for this?

We were recently asked by our statewide catalog if Evergreen could place mutliple holds. Since no one had complained, they assumed it did. I corrected them that it didn't.

Revision history for this message
Jason Boyer (jboyer) wrote :

I made the changes mentioned here and *thought* it did something, but on further testing it still fails when placing multiple holds for the same user. I haven't had a lot of time to fight with it. The short answer is no.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

OK. Thanks!

I think Evergreen may need some changes for this work properly, but I'm not sure. They may have already happened.

I might look into this.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Note: The code for bug 1694058 will make this easier once it goes in.

Changed in ncipserver:
importance: Undecided → Wishlist
status: New → Confirmed
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.