From 7208788d0ee5a935607309ed95193d8b7947b5be Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 17 Dec 2019 14:23:16 +0100 Subject: [PATCH] Bug 24175: Take advantage of Koha::Item->holds Signed-off-by: Jonathan Druart --- C4/SIP/ILS/Transaction/Hold.pm | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/C4/SIP/ILS/Transaction/Hold.pm b/C4/SIP/ILS/Transaction/Hold.pm index 44c1beb1bd..e442787dc5 100644 --- a/C4/SIP/ILS/Transaction/Hold.pm +++ b/C4/SIP/ILS/Transaction/Hold.pm @@ -86,14 +86,10 @@ sub drop_hold { $self->ok(0); return $self; } + my $item = Koha::Items->find({ barcode => $self->{item}->id }); - my $holds = Koha::Holds->search( - { - biblionumber => $item->biblionumber, - itemnumber => $item->itemnumber, - borrowernumber => $patron->borrowernumber - } - ); + my $holds = $item->holds->search({ borrowernumber => $patron->borrowernumber }); + return $self unless $holds->count; $holds->next->cancel; -- 2.11.0