Skip to content

Add support for various decimal separators

bgm requested to merge bgm/lineitemedit:fixDecimals into master

The extension does not work when using money formatted in various languages, such as French (123,95), because of the decimal separator.

This PR re-uses the same technique that was already used for the thousandSeparator, but also a trick from CRM.formatMoney (from js/Common.js in core) to extract the decimal/thousand separators, given that the old settings are getting deprecated now that Brick/Money is used.

Merge request reports