@@ -, +, @@ failures --- Koha/Item.pm | 2 -- t/db_dependent/Holds.t | 10 ++++++++++ 2 files changed, 10 insertions(+), 2 deletions(-) --- a/Koha/Item.pm +++ a/Koha/Item.pm @@ -302,8 +302,6 @@ sub pickup_locations { my $branch_control = C4::Context->preference('HomeOrHoldingBranch'); my $library = $branch_control eq 'holdingbranch' ? $self->holding_branch : $self->home_branch; - #warn $branch_control.' '.$branchitemrule->{holdallowed}.' '.$library->branchcode.' '.$patron->branchcode; - my @libs; if(defined $patron) { return @libs if $branchitemrule->{holdallowed} == 3 && !$library->validate_hold_sibling( {branchcode => $patron->branchcode} ); --- a/t/db_dependent/Holds.t +++ a/t/db_dependent/Holds.t @@ -244,7 +244,12 @@ is( $hold->priority, '6', "Test AlterPriority(), move to bottom" ); my $foreign_biblio = $builder->build_sample_biblio({ itemtype => 'DUMMY' }); my ($foreign_item_bibnum, $foreign_item_bibitemnum, $foreign_itemnumber) = AddItem({ homebranch => $branch_2, holdingbranch => $branch_2 } , $foreign_biblio->biblionumber); +# Cleanup circulation rules $dbh->do('DELETE FROM issuingrules'); +$dbh->do('DELETE FROM branch_item_rules'); +$dbh->do('DELETE FROM default_branch_item_rules'); +$dbh->do('DELETE FROM default_branch_circ_rules'); +$dbh->do('DELETE FROM default_circ_rules'); $dbh->do( q{INSERT INTO issuingrules (categorycode, branchcode, itemtype, reservesallowed, holds_per_record) VALUES (?, ?, ?, ?, ?)}, @@ -565,6 +570,11 @@ subtest 'CanItemBeReserved / holds_per_day tests' => sub { Koha::Holds->search->delete; $dbh->do('DELETE FROM issues'); + $dbh->do('DELETE FROM issuingrules'); + $dbh->do('DELETE FROM branch_item_rules'); + $dbh->do('DELETE FROM default_branch_item_rules'); + $dbh->do('DELETE FROM default_branch_circ_rules'); + $dbh->do('DELETE FROM default_circ_rules'); Koha::Items->search->delete; Koha::Biblios->search->delete; --