From 5fff3388dc23dbd7edb5fcd7916096c1104b54c7 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 21 Oct 2011 10:19:30 -0400 Subject: [PATCH] Fix for Bug 7080 - Clean up interface on fine payment screens - Move some content out of table cells - Improve table markup with , , and - Improve breadcrumb specificity on paycollect.pl - Add clearer messages for different actions (Pay selected fines, pay an individual fine, etc. - Add client-side warning to pay.pl when writing-off all - Correcting terminology: When a verb, "write off," when a noun: "Writeoff." --- .../intranet-tmpl/prog/en/modules/members/pay.tt | 35 ++++- .../prog/en/modules/members/paycollect.tt | 155 +++++++++----------- 2 files changed, 98 insertions(+), 92 deletions(-) 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 6dfd8f9..22c82ce 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt @@ -1,6 +1,18 @@ [% INCLUDE 'doc-head-open.inc' %] Koha › Patrons › Pay Fines for [% borrower.firstname %] [% borrower.surname %] [% INCLUDE 'doc-head-close.inc' %] + [% INCLUDE 'header.inc' %] @@ -30,6 +42,7 @@ + @@ -40,14 +53,21 @@ - + + + + + + + + [% FOREACH account_grp IN accounts %] [% FOREACH line IN account_grp.accountlines %] [% END %] [% END %] - - - - +
Fines & Charges SelAmount Amount Outstanding
Total Due:[% total | format('%.2f') %]
[% IF ( line.amountoutstanding > 0 ) %] - + [% END %] @@ -81,15 +101,12 @@
Total Due:[% total | format('%.2f') %]
- - - + + + Cancel
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt index 08ee909..4637e32 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt @@ -57,7 +57,7 @@ function moneyFormat(textObj) { [% INCLUDE 'header.inc' %] [% INCLUDE 'patron-search.inc' %] - +
@@ -102,50 +102,50 @@ function moneyFormat(textObj) { + +
+ Pay an individual fine - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + +
DescriptionAccount TypeNotify idLevelAmountAmount Outstanding
- [% description %] [% title %] - [% accounttype %][% notify_id %][% notify_level %][% amount | format('%.2f') %][% amountoutstanding | format('%.2f') %]
Total Amount Payable : [% amountoutstanding | format('%.2f') %]
Collect From Patron: +
DescriptionAccount TypeNotify idLevelAmountAmount Outstanding
Total amount payable:[% amountoutstanding | format('%.2f') %]
+ [% description %] [% title %] + [% accounttype %][% notify_id %][% notify_level %][% amount | format('%.2f') %][% amountoutstanding | format('%.2f') %]
+ +
    + +
  1. + - - - - - - - Cancel - - +
  2. +
+
- + [% ELSIF ( writeoff_individual ) %]
+
+ Write off an individual fine @@ -157,33 +157,27 @@ function moneyFormat(textObj) { - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + +
DescriptionAccount TypeNotify idLevelAmountAmount Outstanding
[% description %] [% title %][% accounttype %][% notify_id %][% notify_level %][% amount | format('%.2f') %][% amountoutstanding | format('%.2f') %]
Writeoff This Charge?
- - Cancel -
DescriptionAccount TypeNotify idLevelAmountAmount Outstanding
Total amount to be written off:[% amountoutstanding | format('%.2f') %]
[% description %] [% title %][% accounttype %][% notify_id %][% notify_level %][% amount | format('%.2f') %][% amountoutstanding | format('%.2f') %]
+
+
[% ELSE %] @@ -192,27 +186,22 @@ function moneyFormat(textObj) { - - - - - - - - - - - - - - -
Total Amount Outstanding : [% total | format('%.2f') %]
Collect From Patron: +
+ [% IF ( selected_accts ) %]Pay an amount toward selected fines[% ELSE %]Pay an amount toward all fines[% END %] +
    +
  1. + Total amount outstanding: + [% total | format('%.2f') %] +
  2. +
  3. + -
- - Cancel -
+ + + + [% END %]
-- 1.7.5.4