https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25690 --- Comment #46 from Joonas Kylmälä <joonas.kylmala@helsinki.fi> --- (In reply to Jonathan Druart from comment #44)
- if ( $res->{'itemnumber'} && $res->{'itemnumber'} == $itemnumber && $res->{'priority'} == 0) {
Why did you remove that test?
Thanks for asking. The reason I removed the line is because it equivalent of a reserve being found / attached and now we finally handle all the different cases so doing the check twice is not necessary. If you look at the patch https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67411 (bug 19116) then it should be really clear. Previously the "Reserved" return value had double usage/meaning, firstly it could mean an unattached hold (biblio or item level) or secondly it could have meant an attached hold in transfer. Please see the explanation also in the commit message. -- You are receiving this mail because: You are watching all bug changes.