Summary: | POS doesn't follow CurrencyFormat | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Fines and fees | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
Status: | CLOSED FIXED | QA Contact: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | 1joynelson, jonathan.druart, martin.renvoize, sally.healey, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.05.00
|
|
Circulation function: | |||
Bug Depends on: | 22359, 23354 | ||
Bug Blocks: | 13985 | ||
Attachments: |
Bug 24479: Use $Price filters for '0' amounts
Bug 24479: Use $Price filters for '0' amounts Bug 24479: Use $Price filters for '0' amounts |
Description
Katrin Fischer
2020-01-22 07:31:50 UTC
The points you've made apply equally to the change feature elsewhere and should be fixed there at the same time. I am not sure I undertand - you are using the Price TT plugin in parts of the page, but when you get the confirmation message for change the value is not formatted. <p>The change to give is <b><span id="modal_change">0.00</span></b>.</p> I am not even sure why there is 0.00 in there - and the b span construct seems risky for translation. Created attachment 99903 [details] [review] Bug 24479: Use $Price filters for '0' amounts This patch adds the use of the $Price TT filter to properly format '0' amounts in the POS system. Test plan 1) Change the Currency format preference 2) enter an amount equal to or less than the amount owed into the collected field in the point of sale page and note that on leaving the field it will be formatted as per the system preference. Created attachment 100037 [details] [review] Bug 24479: Use $Price filters for '0' amounts This patch adds the use of the $Price TT filter to properly format '0' amounts in the POS system. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Created attachment 100099 [details] [review] Bug 24479: Use $Price filters for '0' amounts This patch adds the use of the $Price TT filter to properly format '0' amounts in the POS system. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Nice work everyone! Pushed to master for 20.05 not pushed to 19.11.x due to dependencies + <input type="hidden" name="change" value="[% 0 | $Price %]"/> I think this is not correct, $Price should not be used for hidden inputs. |