Bug 17078

Summary: Format fines on opac-account.pl
Product: Koha Reporter: Marc Véron <veron>
Component: OPACAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: f.demians, julian.maurice, kyle
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17076
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 17078 - Format fines on opac-account.pl
[SIGNED-OFF] Bug 17078 - Format fines on opac-account.pl
Bug 17078 - Format fines on opac-account.pl

Description Marc Véron 2016-08-08 12:01:46 UTC
Format fines on page "your fines" (opac-account.pl) following syspref CurrencyFormat

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)
Comment 1 Marc Véron 2016-08-08 12:11:20 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2016-08-08 13:11:42 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-08-09 09:24:03 UTC
Created attachment 54185 [details] [review]
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>
Comment 4 Kyle M Hall 2016-08-10 14:10:37 UTC
Pushed to master for Koha 16.11, thanks Marc!
Comment 5 Frédéric Demians 2016-08-23 10:09:56 UTC
Pushed in 16.05. Will be in 16.05.03.
Comment 6 Julian Maurice 2016-08-24 09:26:42 UTC
Pushed to 3.22.x, will be in 3.22.10