Attachment 54185 Details for Bug 17078 – Bug 17078 - Format fines on opac-account.pl

[patch] Bug 17078 - Format fines on opac-account.pl

Bug-17078---Format-fines-on-opac-accountpl.patch (text/plain), 2.97 KB, created by Jonathan Druart on 2016-08-09 09:24:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-08-09 09:24:03 UTC

Size: 2.97 KB

patch

obsolete

>From e892f982f984447e72ce2577db75724a793edddf Mon Sep 17 00:00:00 2001
>From: Marc <veron@veron.ch>
>Date: Mon, 8 Aug 2016 14:04:44 +0200
>Subject: [PATCH] Bug 17078 - Format fines on opac-account.pl
>
>To verify:
>- Log in to OPAC as a user who has fines or credits
>- Go to page "your fines"
>- Verify that values for "Fine amount", "Amount outstanding"
>  and "Total due" are not formatted as defined in syspref CurrencyFormat
>  (e.g. for FR)
>
>To test:
>- Apply patch
>- Repeat steps above
>- Verify that values are formatted as appropriate with different
>  settings for syspref CurrencyFormat
>
>Note: This patch does not force text alignment to the right.
>      Text alignement can be done using syspref OPACUserCSS
>      (td.sum, td.credit, td.debit)
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
>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 289acfd..e38b10d 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>@@ -1,5 +1,6 @@
> [% USE Koha %]
> [% USE KohaDates %]
>+[% USE Price %]
> [% SET ENABLE_OPAC_PAYMENTS = Koha.Preference('EnablePayPalOpacPayments') %]
> [% SET DISPLAY_PAYMENT_BLOCK = 0 %]
> 
>@@ -84,7 +85,7 @@
>                                     [%- SET COLSPAN = 3 -%]
>                                 [%- END -%]
>                                 <th class="sum" colspan="[% COLSPAN %]">Total due</th>
>-                                <td class="sum">[% total %]</td>
>+                                <td class="sum">[% total | $Price %]</td>
>                             </tr>
>                             </tfoot>
> 
>@@ -127,8 +128,8 @@
>                                           [%- IF ACCOUNT_LINE.description %], [% ACCOUNT_LINE.description %][% END %]
>                                           [% IF ACCOUNT_LINE.title %]([% ACCOUNT_LINE.title %])[% END %]
>                                         </td>
>-                                        [% IF ( ACCOUNT_LINE.amountcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amount %]</td>
>-                                        [% IF ( ACCOUNT_LINE.amountoutstandingcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amountoutstanding %]</td>
>+                                        [% IF ( ACCOUNT_LINE.amountcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amount | $Price %]</td>
>+                                        [% IF ( ACCOUNT_LINE.amountoutstandingcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amountoutstanding | $Price %]</td>
>                                     </tr>
>                                 [% END %]
>                             </tbody>
>-- 
>2.8.1

Actions: View | Diff | Splinter Review

Attachments on bug 17078: 54155 | 54160 | 54185