10 Jun
2013
10 Jun
'13
6:37 p.m.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9394 --- Comment #37 from Kyle M Hall <kyle.m.hall@gmail.com> --- (In reply to comment #36)
I've started reviewing this. One thing that jumped out -- the QA follow-up (re?)introduces a change in the behavior of AlterPriority that should get a test case. I'll deal with that with a follow-up patch myself unless I find other reason to bounce back to QA.
Good catch! I'm re-evaluating it right now and I don't think it's going to affect anything. Koha does not expose canceled reserves iirc, so a scenario where a canceled reserve has it's priority altered shouldn't come up. That may be why Jonathan added the check. -- You are receiving this mail because: You are watching all bug changes.