From c09db113208f77ac9c2edcbb8c993eea04c452db Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 26 Dec 2022 15:27:02 -0300 Subject: [PATCH] Bug 32529: Make Koha::Item->safe_to_delete consider 'P' holds as found Currently, the Koha::Item->safe_to_delete method checks (among other things) if there are existing waiting or in-transit holds. It seems that 'In processing' should block item deletion as well. This patch implements that. To test: 1. Apply up to the regression tests 2. Run: $ kshell k$ prove t/db_dependent/Koha/Item.t => FAIL: Boo! Tests fail! 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! 5. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 0e8f28b94212da131a2a289c500cd823dbc4a5e7) Signed-off-by: Jacob O'Mara --- Koha/Item.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Koha/Item.pm b/Koha/Item.pm index e508e7eb55..1174519483 100644 --- a/Koha/Item.pm +++ b/Koha/Item.pm @@ -301,7 +301,7 @@ sub safe_to_delete { # check it doesn't have a waiting reserve $error = "book_reserved" - if $self->holds->search( { found => [ 'W', 'T' ] } )->count; + if $self->holds->filter_by_found->count; $error = "linked_analytics" if C4::Items::GetAnalyticsCount( $self->itemnumber ) > 0; -- 2.39.5