http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4045 --- Comment #68 from Kyle M Hall <kyle@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.