Bug 9818

Summary: "Overdue Fines Cap ($)" should not use $ (I18N)
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: I18N/L10NAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: normal    
Priority: P5 - low CC: chris, f.demians, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9818: Fixing capitalization and remove dollar sign
[SIGNED-OFF] Bug 9818: Fixing capitalization and remove dollar sign
Bug 9818: Fixing capitalization and remove dollar sign
Bug 9818: Fixing capitalization and remove dollar sign
Bug 9818: Fixing capitalization and remove dollar sign

Description Katrin Fischer 2013-03-14 10:03:42 UTC
While it would be good to have a plugin to allow showing the active currency's symbol or code in various places, we should avoid hardcoding a currency symbol in the template for now.
Comment 1 Katrin Fischer 2013-03-14 10:04:14 UTC
Sorry - string appears on the circulation rules page!
Comment 2 Katrin Fischer 2013-04-15 04:27:31 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2013-04-15 14:20:12 UTC Comment hidden (obsolete)
Comment 4 Chris Cormack 2013-04-15 21:10:29 UTC Comment hidden (obsolete)
Comment 5 Chris Cormack 2013-04-15 21:15:36 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2013-04-25 08:27:52 UTC
String patch, passed QA.
Comment 7 Jonathan Druart 2013-04-25 08:28:07 UTC
Created attachment 17669 [details] [review]
Bug 9818: Fixing capitalization and remove dollar sign

Overdue Fines Cap ($) > Overdue fines cap (amount)

To test:
Go to administration > circulation and fine rules
Check column label Overdue fines cap now follows capitalization rules
and has no dollar sign.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 8 Jared Camins-Esakov 2013-04-26 01:53:58 UTC
This patch has been pushed to master and 3.12.x.
Comment 9 Chris Cormack 2013-04-27 21:36:05 UTC
Pushed to 3.10.x will be in 3.10.6