https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23964
--- Comment #13 from Nick Clemens <[email protected]> --- (In reply to Jonathan Druart from comment #11) > 725 my $hold_1 = place_item_hold( $patron, $item, $library, $priority ); > 726 $hold = place_item_hold( $patron_2, $item, $library, $priority ); > 727 is( $hold->priority, 1, 'If ReservesNeedReturns is 0 but item > already on hold priority must be set to 1' ); > > What about $hold_1->priority? Should not we have $hold_1->priority == 1 > instead of $hold->priority == 1? The first hold should get set to waiting, the next hold on the item doesn't because there is a hold already on the item > 737 isnt( $hold->priority, 0, 'If ReservesNeedReturns is 0 but item in > transit the hold must not be set to waiting' ); > > Test's description does not match the test. I do not understand, I place a transfer before this test, the transfer means the hold is not waiting. Changed to 'is' '1' in any case -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
