[Koha-bugs] [Bug 4045] No check for maximum number of allowed holds from OPAC.

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4045

--- Comment #68 from Kyle M Hall <kyle at bywatersolutions.com> ---
I see now that it did work for multi-holds in the past. Considering I started
this patch 2 years ago I'm not surprised I forgot ; )

(In reply to Christopher Brannon from comment #67)
> (In reply to Kyle M Hall from comment #66)
> > I don't think this should fail qa for not being a complete fix. A partial
> > fix is better than no fix at all!
> > 
> > Kyle
> > 
> > (In reply to Christopher Brannon from comment #65)
> > > Got the sandbox to work this time around.
> > > 
> > > The limit works great for individually placed holds, but still gives no
> > > warning and places holds over limit if holding multiple items.
> > > 
> > > Christopher
> 
> I respectfully disagree.  You placed the multi-hold as a test condition.
> 
> > 9) Attempt to place 3 or more holds as a multi-hold
> 
> The patch does not pass your test.  If the patch is only to test items
> placed on hold one at a time, then it works just fine.  I don't think this
> should pass, based on your parameters.
> 
> Christopher

-- 
You are receiving this mail because:
You are watching all bug changes.