Summary: | Currency page shows perl error under some circumstances | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | jonathan.druart, m.de.rooy, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Bug 13071: format_date should not be called in currency.pl
[SIGNED OFF] Bug 13071: format_date should not be called in currency.pl Bug 13071: format_date should not be called in currency.pl |
Description
Katrin Fischer
2014-10-10 20:17:13 UTC
Created attachment 32205 [details] [review] Bug 13071: format_date should not be called in currency.pl The TT plugin is used in the template, it is useless and can cause problems to format the dates before sending them to the template. Created attachment 32212 [details] [review] [SIGNED OFF] Bug 13071: format_date should not be called in currency.pl The TT plugin is used in the template, it is useless and can cause problems to format the dates before sending them to the template. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Fixes the problem I was seeing before, didn't find any regressions. Created attachment 32474 [details] [review] Bug 13071: format_date should not be called in currency.pl The TT plugin is used in the template, it is useless and can cause problems to format the dates before sending them to the template. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Fixes the problem I was seeing before, didn't find any regressions. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Checked all three settings of dateformat. Looks fine to me. Patch pushed to master. Thanks Jonathan! |