From b3f525497ba3fb21bd4f56bd5dea00fd425366fb Mon Sep 17 00:00:00 2001
From: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Date: Mon, 23 Mar 2026 13:22:04 -0400
Subject: [PATCH] Bug 41386: (QA follow-up) Make sure charge is defined

Check that $charge is defined before comparing, to avoid "argument isn't
numeric" error if no charge is defined.

Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
(cherry picked from commit 699ba06eb70d3488fba0ba47f406ffe4fe96f4f9)
Signed-off-by: Jacob O'Mara <Jacob.omara@openfifth.co.uk>
Signed-off-by: Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz>
---
 Koha/Hold.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Koha/Hold.pm b/Koha/Hold.pm
index 091d6fe28a..a69524dd59 100644
--- a/Koha/Hold.pm
+++ b/Koha/Hold.pm
@@ -785,7 +785,7 @@ sub cancel {
                         type       => 'RESERVE_EXPIRED',
                         item_id    => $self->itemnumber
                     }
-                ) if $charge > 0;
+                ) if $charge && $charge > 0;
             }
 
             C4::Log::logaction( 'HOLDS', 'CANCEL', $self->reserve_id, $self, undef, $original )