From 2e4459ea69f9db2baf7258d0e410ca5871c3e180 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 7 Apr 2026 14:49:33 -0400
Subject: [PATCH] Bug 41728: (QA follow-up) Declare issue and patron variables
 together
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

There seems to be no reason to declare $patron near the start of the sub
while $issue is declared where they are both populated.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
---
 C4/Circulation.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 5b6b2d2f9b..3f342a3412 100644
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -2442,7 +2442,7 @@ sub AddReturn {
 
     # Extract context objects
     my $issue = $availability->context->{checkout};
-    $patron = $availability->context->{patron};
+    $patron   = $availability->context->{patron};
 
     # Set NotIssued message if item not checked out
     if ( !$issue ) {