From f916269465df99818fb15c108820d3b666968d39 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Fri, 31 Jul 2020 11:14:20 -0300
Subject: [PATCH] Bug 26114: ILL should mark status=RET only if a return
 happened

The original code assumed every codepath should be trated as a
successful check-in. Such is not the case and the regression tests
verify that the code acknowledges this.

To test:
1. Apply the regression tests patch
2. Run:
   $ kshell
  k$ prove t/db_dependent/Illrequests.t
=> FAIL: Tests fail
3. Apply this patch
4. Repeat 2
=> SUCCESS: Tests pass!
5. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 C4/Circulation.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 629728666f..f530d92a72 100644
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -2201,7 +2201,7 @@ sub AddReturn {
 
     # Check if this item belongs to a biblio record that is attached to an
     # ILL request, if it is we need to update the ILL request's status
-    if (C4::Context->preference('CirculateILL')) {
+    if ( $doreturn and C4::Context->preference('CirculateILL')) {
         my $request = Koha::Illrequests->find(
             { biblio_id => $item->biblio->biblionumber }
         );
-- 
2.39.5