From 77ce011382a27325d63692c4744ff090b0ce41f0 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Wed, 4 Dec 2019 18:50:41 +0000
Subject: [PATCH] Bug 24175: Use itemnumber when searching for holds in SIP
drop_hold
To test:
1 - place a hold for a patron
2 - Get SIP running
https://wiki.koha-community.org/wiki/Koha_SIP2_server_setup
3 - edit the sip_cli_emulator.pl for hold_mode from '+' to '-'
4 - Attempt to cancel hold
5 - It fails
6 - Apply patch
7 - Try again
8 - It succeeds
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
C4/SIP/ILS/Transaction/Hold.pm | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/C4/SIP/ILS/Transaction/Hold.pm b/C4/SIP/ILS/Transaction/Hold.pm
index 825e89b9db..44c1beb1bd 100644
--- a/C4/SIP/ILS/Transaction/Hold.pm
+++ b/C4/SIP/ILS/Transaction/Hold.pm
@@ -87,11 +87,10 @@ sub drop_hold {
return $self;
}
my $item = Koha::Items->find({ barcode => $self->{item}->id });
-
my $holds = Koha::Holds->search(
{
biblionumber => $item->biblionumber,
- itemnumber => $self->{item}->id,
+ itemnumber => $item->itemnumber,
borrowernumber => $patron->borrowernumber
}
);
--
2.11.0