diff --git a/C4/HoldsQueue.pm b/C4/HoldsQueue.pm index e2666aa4ab..d4380ccfa8 100644 --- a/C4/HoldsQueue.pm +++ b/C4/HoldsQueue.pm @@ -383,12 +383,12 @@ sub _checkHoldPolicy { my $hold_fulfillment_policy = $item->{hold_fulfillment_policy}; return 0 if $hold_fulfillment_policy eq 'holdgroup' && !$library->validate_hold_sibling({branchcode => $request->{branchcode}}); - return 0 if $hold_fulfillment_policy eq 'homebranch' && $request->{branchcode} ne $item->$hold_fulfillment_policy; - return 0 if $hold_fulfillment_policy eq 'holdingbranch' && $request->{branchcode} ne $item->$hold_fulfillment_policy; + return 0 if $hold_fulfillment_policy eq 'homebranch' && $request->{branchcode} ne $item->{$hold_fulfillment_policy}; + return 0 if $hold_fulfillment_policy eq 'holdingbranch' && $request->{branchcode} ne $item->{$hold_fulfillment_policy}; my $patronLibrary = Koha::Libraries->find($request->{borrowerbranch}); - return 0 if $hold_fulfillment_policy eq 'patrongroup' && !patronLibrary->validate_hold_sibling({branchcode => $request->{branchcode}}); + return 0 if $hold_fulfillment_policy eq 'patrongroup' && !$patronLibrary->validate_hold_sibling({branchcode => $request->{branchcode}}); return 1; diff --git a/Koha/Library.pm b/Koha/Library.pm index b69fdaaeab..fc2f4f522e 100644 --- a/Koha/Library.pm +++ b/Koha/Library.pm @@ -240,6 +240,9 @@ Return if given library is a valid hold group member sub validate_hold_sibling { my ( $self, $params ) = @_; + + return 1 if $params->{branchcode} eq $self->id; + my @hold_libraries = $self->get_hold_libraries; foreach (@hold_libraries) {