Attachment 85399 Details for Bug 12166 – Bug 12166: Improve display of Reserve charges in patron account

[patch] Bug 12166: Improve display of Reserve charges in patron account

Bug-12166-Improve-display-of-Reserve-charges-in-pa.patch (text/plain), 11.18 KB, created by Owen Leonard on 2019-02-20 17:48:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2019-02-20 17:48:03 UTC

Size: 11.18 KB

patch

obsolete

>From a87302acbcd50861f169c7dd0333c1a2b456ea7e Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 20 Feb 2019 17:14:22 +0000
>Subject: [PATCH] Bug 12166: Improve display of Reserve charges in patron
> account
>
>This patch makes changes to the way hold fees are recorded and displayed
>in the OPAC and staff client:
>
> - No English strings should be stored in the accountline description.
> - The accounttype code should be used to display what kind of charge it
> is.
>
>To test, apply the patch and go to the staff client:
>
> - Configure a patron category to have a hold fee.
> - Place one or more holds for a patron in that category.
> - On the patron's record, view Accounting -> Make a payment
> - There should be a separate column for "Account type" showing "Hold
> fee" for the hold charges.
> - There should be no "Res" or "Reserve Charge" text in the
> description column--just the title.
> - Switch to the "Transactions" tab. The display should be similar.
> - In the OPAC, log in as a patron who has hold fees on their account.
> - View the "your fines" page to confirm that the information is
> displayed correctly there as well.
>
> - Place a hold for a patron who incurs hold charges.
> - Log in to the self-checkout module as that patron and check out an
> item which will fulfill that hold.
> - The correct fine information should be saved to accountlines: A "Res"
> type with a description consisting only of the title.
>---
> C4/Reserves.pm | 2 +-
> .../intranet-tmpl/prog/en/includes/accounts.inc | 1 +
> .../prog/en/modules/members/boraccount.tt | 9 +++---
> .../intranet-tmpl/prog/en/modules/members/pay.tt | 32 ++++------------------
> .../opac-tmpl/bootstrap/en/modules/opac-account.tt | 12 +++++---
> opac/sco/sco-main.pl | 2 +-
> 6 files changed, 22 insertions(+), 36 deletions(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 974168f..6a5d290 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -584,7 +584,7 @@ sub ChargeReserveFee {
> accountno => $nextacctno,
> date => dt_from_string(),
> amount => $fee,
>- description => "Reserve Charge - $title",
>+ description => $title,
> accounttype => 'Res',
> amountoutstanding => $fee,
> branchcode => $branchcode
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>index cf38823..6e8c2ba 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>@@ -21,6 +21,7 @@
> [%- CASE 'WO' -%]Writeoff
> [%- CASE 'C' -%]Credit
> [%- CASE 'CR' -%]Credit
>+ [%- CASE 'Res' -%]Hold fee
> [%- CASE -%][% account.accounttype | html %]
> [%- END -%]
> [%- END -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>index bd54ec7..d4d3a8e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>@@ -44,6 +44,7 @@
> <thead>
> <tr>
> <th class="title-string">Date</th>
>+ <th>Account type</th>
> <th>Description of charges</th>
> <th>Home library</th>
> <th>Note</th>
>@@ -58,10 +59,10 @@
> 
> <tr>
> <td>[% account.date |$KohaDates %]</td>
>+ <td>[% PROCESS account_type_description account=account %]</td>
> <td>
>- [% PROCESS account_type_description account=account %]
>- [%- IF account.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', account.payment_type) | html %][% END %]
>- [%- IF account.description %], [% account.description | html %][% END %]
>+ [%- IF account.payment_type %][% AuthorisedValues.GetByCode('PAYMENT_TYPE', account.payment_type) | html %][% END %]
>+ [%- IF account.description %][% account.description | html %][% END %]
> &nbsp;[% IF ( account.itemnumber ) %]<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% account.item.biblionumber | uri %]&amp;itemnumber=[% account.itemnumber | uri %]">[% account.item.biblio.title | html %]</a>[% END %]</td>
> <td>[% IF account.itemnumber %][% Branches.GetName( account.item.homebranch ) | html %][% END %]</td>
> <td>[% account.note | html_line_break %]</td>
>@@ -83,7 +84,7 @@
> [% END %]
> <tfoot>
> <tr>
>- <td colspan="5">Total due</td>
>+ <td colspan="6">Total due</td>
> [% IF ( totalcredit ) %]
> <td class="credit" style="text-align: right;">[% total | $Price %]</td>
> [% ELSE %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
>index 08350ab..cd1d2c9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
>@@ -7,6 +7,7 @@
> [% USE ColumnsSettings %]
> [% USE KohaDates %]
> [% SET footerjs = 1 %]
>+[% PROCESS 'accounts.inc' %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; Patrons &rsaquo; Make a payment for [% patron.firstname | html %] [% patron.surname | html %]</title>
> [% INCLUDE 'doc-head-close.inc' %]
>@@ -44,10 +45,10 @@
> <tr>
> <th class="NoSort">&nbsp;</th>
> <th class="NoSort">Actions</th>
>+ <th>Account type</th>
> <th>Description</th>
> <th class="title-string">Date</th>
> <th class="NoSort">Payment note</th>
>- <th>Account type</th>
> <th>Amount</th>
> <th>Amount outstanding</th>
> </tr>
>@@ -77,30 +78,10 @@
> <input type="hidden" name="borrowernumber[% line.accountlines_id | html %]" value="[% line.borrowernumber | html %]" />
> </td>
> <td>
>- [% SWITCH line.accounttype %]
>- [% CASE 'Pay' %]Payment, thanks
>- [% CASE 'Pay00' %]Payment, thanks (cash via SIP2)
>- [% CASE 'Pay01' %]Payment, thanks (VISA via SIP2)
>- [% CASE 'Pay02' %]Payment, thanks (credit card via SIP2)
>- [% CASE 'N' %]New card
>- [% CASE 'F' %]Fine
>- [% CASE 'A' %]Account management fee
>- [% CASE 'M' %]Sundry
>- [% CASE 'L' %]Lost item
>- [% CASE 'W' %]Writeoff
>- [% CASE 'FU' %]Accruing fine
>- [% CASE 'HE' %]Hold waiting too long
>- [% CASE 'Rent' %]Rental fee
>- [% CASE 'FOR' %]Forgiven
>- [% CASE 'LR' %]Lost item fee refund
>- [% CASE 'PF' %]Processing fee
>- [% CASE 'PAY' %]Payment
>- [% CASE 'WO' %]Writeoff
>- [% CASE 'C' %]Credit
>- [% CASE 'CR' %]Credit
>- [% CASE %][% line.accounttype | html %]
>- [%- END -%]
>- [%- IF line.description %], [% line.description | html %][% END %]
>+ [% PROCESS account_type_description account=line %]
>+ </td>
>+ <td>
>+ [%- IF line.description %][% line.description | html %][% END %]
> [% IF line.title %]([% line.title | html %])[% END %]
> </td>
> <td>
>@@ -110,7 +91,6 @@
> <a href="#" class="add-note" data-accountlines_id="[% line.accountlines_id | html %]"> Add note</a>
> <input type="text" size="10" name="payment_note_[% line.accountlines_id | html %]" value="" /> <a href="#" class="cancel-note"></a>
> </td>
>- <td>[% line.accounttype | html %]</td>
> <td class="debit" style="text-align: right;">[% line.amount | $Price %]</td>
> <td class="debit" style="text-align: right;">[% line.amountoutstanding | $Price %]</td>
> </tr>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>index 23229a0..e4ed1a6 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>@@ -75,6 +75,7 @@
> <tr>
> [% IF ENABLE_OPAC_PAYMENTS %]<th>&nbsp;</th>[% END %]
> <th class="title-string">Date</th>
>+ <th>Type</th>
> <th>Description</th>
> <th>Fine amount</th>
> <th>Amount outstanding</th>
>@@ -84,9 +85,9 @@
> <tfoot>
> <tr>
> [%- IF ENABLE_OPAC_PAYMENTS -%]
>- [%- SET COLSPAN = 4 -%]
>+ [%- SET COLSPAN = 5 -%]
> [%- ELSE -%]
>- [%- SET COLSPAN = 3 -%]
>+ [%- SET COLSPAN = 4 -%]
> [%- END -%]
> <th class="sum" colspan="[% COLSPAN | html %]">Total due</th>
> <td class="sum">[% total | $Price %]</td>
>@@ -128,10 +129,13 @@
> [% CASE 'WO' %]Writeoff
> [% CASE 'C' %]Credit
> [% CASE 'CR' %]Credit
>+ [%- CASE 'Res' -%]Hold fee
> [% CASE %][% ACCOUNT_LINE.accounttype | html %]
> [%- END -%]
>- [%- IF ACCOUNT_LINE.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', ACCOUNT_LINE.payment_type, 1) | html %][% END %]
>- [%- IF ACCOUNT_LINE.description %], [% ACCOUNT_LINE.description | html %][% END %]
>+ </td>
>+ <td>
>+ [%- IF ACCOUNT_LINE.payment_type %][% AuthorisedValues.GetByCode('PAYMENT_TYPE', ACCOUNT_LINE.payment_type, 1) | html %][% END %]
>+ [%- IF ACCOUNT_LINE.description %][% ACCOUNT_LINE.description | html %][% END %]
> [% IF ACCOUNT_LINE.title %]([% ACCOUNT_LINE.title | html %])[% END %]
> </td>
> [% IF ( ACCOUNT_LINE.amountcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amount | $Price %]</td>
>diff --git a/opac/sco/sco-main.pl b/opac/sco/sco-main.pl
>index 841b59b..ec19954 100755
>--- a/opac/sco/sco-main.pl
>+++ b/opac/sco/sco-main.pl
>@@ -229,7 +229,7 @@ elsif ( $patron and $op eq "checkout" ) {
> {
> borrowernumber => $borrower->{borrowernumber},
> accounttype => 'Res',
>- description => 'Reserve Charge - ' . $item->biblio->title,
>+ description => $item->biblio->title,
> date => $dtf->format_date(dt_from_string)
> }
> )->count,
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 12166: 85399 | 87423 | 87472 | 87473 | 87475 | 87478 | 87479 | 87520 | 87521 | 87584 | 87585 | 87586 | 87587