|
Lines 709-721
subtest 'ReservesNeedReturns' => sub {
Link Here
|
| 709 |
|
709 |
|
| 710 |
$item->onloan('2010-01-01')->store; |
710 |
$item->onloan('2010-01-01')->store; |
| 711 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
711 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
| 712 |
isnt( $hold->priority, 0, 'If ReservesNeedReturns is 0 but item onloan priority must not be set to 0' ); |
712 |
is( $hold->priority, 1, 'If ReservesNeedReturns is 0 but item onloan priority must be set to 1' ); |
| 713 |
$hold->delete; |
713 |
$hold->delete; |
| 714 |
|
714 |
|
| 715 |
t::lib::Mocks::mock_preference('AllowHoldsOnDamagedItems', 0); # '0' means damaged holds not allowed |
715 |
t::lib::Mocks::mock_preference('AllowHoldsOnDamagedItems', 0); # '0' means damaged holds not allowed |
| 716 |
$item->onloan(undef)->damaged(1)->store; |
716 |
$item->onloan(undef)->damaged(1)->store; |
| 717 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
717 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
| 718 |
isnt( $hold->priority, 0, 'If ReservesNeedReturns is 0 but item damaged and not allowed holds on damaged items priority must not be set to 0' ); |
718 |
is( $hold->priority, 1, 'If ReservesNeedReturns is 0 but item damaged and not allowed holds on damaged items priority must be set to 1' ); |
| 719 |
$hold->delete; |
719 |
$hold->delete; |
| 720 |
t::lib::Mocks::mock_preference('AllowHoldsOnDamagedItems', 1); # '0' means damaged holds not allowed |
720 |
t::lib::Mocks::mock_preference('AllowHoldsOnDamagedItems', 1); # '0' means damaged holds not allowed |
| 721 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
721 |
$hold = place_item_hold( $patron, $item, $library, $priority ); |
|
Lines 724-734
subtest 'ReservesNeedReturns' => sub {
Link Here
|
| 724 |
|
724 |
|
| 725 |
my $hold_1 = place_item_hold( $patron, $item, $library, $priority ); |
725 |
my $hold_1 = place_item_hold( $patron, $item, $library, $priority ); |
| 726 |
$hold = place_item_hold( $patron_2, $item, $library, $priority ); |
726 |
$hold = place_item_hold( $patron_2, $item, $library, $priority ); |
| 727 |
isnt( $hold->priority, 0, 'If ReservesNeedReturns is 0 but item already on hold priority must not be set to 0' ); |
727 |
is( $hold->priority, 1, 'If ReservesNeedReturns is 0 but item already on hold priority must be set to 1' ); |
| 728 |
$hold->delete; |
728 |
$hold->delete; |
| 729 |
$hold_1->delete; |
729 |
$hold_1->delete; |
| 730 |
|
730 |
|
| 731 |
$hold->delete; |
|
|
| 732 |
$builder->build({ source => "Branchtransfer", value => { |
731 |
$builder->build({ source => "Branchtransfer", value => { |
| 733 |
itemnumber => $item->itemnumber, |
732 |
itemnumber => $item->itemnumber, |
| 734 |
datearrived => undef, |
733 |
datearrived => undef, |
| 735 |
- |
|
|