From f07239c5c31a8759c930464935a2c67e608bf769 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Thu, 15 Nov 2018 18:56:00 +0000 Subject: [PATCH] Bug 21844: Include itemcallnumber in lost ifne description To test: 1 - Make sure WhenLostChargeReplacementFee is set to charge 2 - Find an item with a replacement cost (or default) and a call number 3 - Checkout the item to a patron 4 - Mark the item lost 5 - View the fine - the description includes title and barcode 6 - Apply patch 7 - Check the item in 8 - Check it out again 9 - Mark it lost 10 - Note the fine inclues the callnumber --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 12460efbf0..19b85b8108 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -3697,7 +3697,7 @@ sub LostItem{ defined($fix) or warn "_FixOverduesOnReturn($borrowernumber, $itemnumber...) failed!"; # zero is OK, check defined } if (C4::Context->preference('WhenLostChargeReplacementFee')){ - C4::Accounts::chargelostitem($borrowernumber, $itemnumber, $issues->{'replacementprice'}, "Lost Item $issues->{'title'} $issues->{'barcode'}"); + C4::Accounts::chargelostitem($borrowernumber, $itemnumber, $issues->{'replacementprice'}, "Lost Item $issues->{'title'} $issues->{'barcode'} $issues->{'itemcallnumber'}"); #FIXME : Should probably have a way to distinguish this from an item that really was returned. #warn " $issues->{'borrowernumber'} / $itemnumber "; } -- 2.11.0