From 4d2df6f9859ddc1f1d96008a6b3373380f0913d7 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 5 Feb 2024 14:44:46 -0500 Subject: [PATCH] Bug 35997: Cancelling a hold should remove the hold from the queue Test plan: 1) Place a hold for a record for a patron 2) Build the holds queue 3) Note the holds queue contains a match for that patron and record 4) Cancel the holds queue 5) Reload the holds queue page, note the match for that patron and record is gone! Signed-off-by: Brendan Lawlor Signed-off-by: Victor Grousset/tuxayo Signed-off-by: Katrin Fischer --- Koha/Hold.pm | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/Koha/Hold.pm b/Koha/Hold.pm index 0224d019b7..a74ed95b58 100644 --- a/Koha/Hold.pm +++ b/Koha/Hold.pm @@ -706,6 +706,21 @@ sub cancel { $self->cancellation_reason( $params->{cancellation_reason} ); $self->store(); + my $dbh = $self->_result->result_source->schema->storage->dbh; + $dbh->do( + q{ + DELETE q, t + FROM tmp_holdsqueue q + INNER JOIN hold_fill_targets t + ON q.borrowernumber = t.borrowernumber + AND q.biblionumber = t.biblionumber + AND q.itemnumber = t.itemnumber + AND q.item_level_request = t.item_level_request + AND q.holdingbranch = t.source_branchcode + WHERE t.reserve_id = ? + }, undef, $self->id + ); + if ( $params->{cancellation_reason} ) { my $letter = C4::Letters::GetPreparedLetter( module => 'reserves', -- 2.39.5