http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4045
--- Comment #67 from Christopher Brannon <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
