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>

diff --git a/C4/Accounts.pm b/C4/Accounts.pm
index 34dcede..c43741b 100644 (file)
--- 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;
 }