From f5d9c9d6bbb65499206cab1d9a03048bbbe889b6 Mon Sep 17 00:00:00 2001
From: Michael Hafen <mdhafen@tech.washk12.org>
Date: Tue, 2 Sep 2008 13:57:09 -0600
Subject: [PATCH] bug: chargelostitem() didn't unset onloan on the item record.

I don't think this is left set intentionally.

If an item is set lost and it's on loan it's marked returned, but it
wasn't set as not on loan.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
---
 C4/Accounts.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/C4/Accounts.pm b/C4/Accounts.pm
index 34dcedee28..c43741bf49 100644
--- a/C4/Accounts.pm
+++ b/C4/Accounts.pm
@@ -313,6 +313,8 @@ sub chargelostitem{
         #FIXME : Should probably have a way to distinguish this from an item that really was returned.
         warn " $issues->{'borrowernumber'}  /  $itemnumber ";
         C4::Circulation::MarkIssueReturned($issues->{borrowernumber},$itemnumber);
+	#  Shouldn't MarkIssueReturned do this?
+        ModItem({ onloan => undef }, undef, $itemnumber);
     }
     $sth->finish;
 }
-- 
2.39.2