@@ -, +, @@ if they are! ensure there are no regressions. - This patch would be nicer with a regression test. - Also checked that returning the item removes lost status and onloan still. - Tried to test with --mark-returned, but couldn't get it to return my item neither with nor without the patch. (see comment on bug report) --- C4/Circulation.pm | 2 ++ C4/Items.pm | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) --- a/C4/Circulation.pm +++ a/C4/Circulation.pm @@ -2073,6 +2073,8 @@ sub MarkIssueReturned { WHERE borrowernumber = ? AND itemnumber = ?"); $sth_del->execute($borrowernumber, $itemnumber); + + ModItem( { 'onloan' => undef }, undef, $itemnumber ); } =head2 _debar_user_on_return --- a/C4/Items.pm +++ a/C4/Items.pm @@ -540,8 +540,6 @@ sub ModItem { $item->{'itemnumber'} = $itemnumber or return; - $item->{onloan} = undef if $item->{itemlost}; - my @fields = qw( itemlost withdrawn ); # Only call GetItem if we need to set an "on" date field --