From 3be2821ee4c1be4415ef0f5eace0842526084c92 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Fri, 22 Mar 2019 16:28:52 +0000 Subject: [PATCH] Bug 22563: Update lost handling to use accountline.status Signed-off-by: Martin Renvoize --- C4/Circulation.pm | 4 ++-- koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc | 2 +- koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 82010ddfb7..6e4f7c0d4e 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2426,8 +2426,8 @@ sub _FixAccountForLostAndReturned { $credit->apply( { debits => $accountlines->reset } ); } - # Manually set the accounttype - $accountline->discard_changes->accounttype('LR'); + # Update the account status + $accountline->discard_changes->status('RETURNED'); $accountline->store; ModItem( { paidfor => '' }, undef, $itemnumber, { log_action => 0 } ); diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc index b884f85135..41fac2d5e6 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc @@ -8,7 +8,7 @@ [%- CASE 'OVERDUE' -%]Fine[%- PROCESS account_status_description account=account -%] [%- CASE 'A' -%]Account management fee [%- CASE 'M' -%]Sundry - [%- CASE 'LOST' -%]Lost item + [%- CASE 'LOST' -%]Lost item[%- PROCESS account_status_description account=account -%] [%- CASE 'W' -%]Writeoff[%- PROCESS account_status_description account=account -%] [%- CASE 'HE' -%]Hold waiting too long [%- CASE 'Rent' -%]Rental fee diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc index 639c3c686a..446f78d338 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc @@ -38,7 +38,7 @@ [% CASE 'OVERDUE' %]Fine[%- PROCESS account_status_description account=account -%] [% CASE 'A' %]Account management fee [% CASE 'M' %]Sundry - [% CASE 'LOST' %]Lost item + [% CASE 'LOST' %]Lost item[%- PROCESS account_status_description account=account -%] [% CASE 'W' %]Writeoff[%- PROCESS account_status_description account=account -%] [% CASE 'HE' %]Hold waiting too long [% CASE 'Rent' %]Rental fee -- 2.20.1