From a5eaa7a37d38f4de623918d498d56a2e7bd2bcec 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 fine 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 includes the callnumber Signed-off-by: Pierre-Marc Thibault Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index fc1379d186..dcb18b3401 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -3695,7 +3695,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.39.5