projects / civicrm-core.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: faa0e6a) | patch
dev/core#2493 Stop attempting to format money in the processor class
authoreileen <emcnaughton@wikimedia.org>
Mon, 12 Apr 2021 03:23:07 +0000 (15:23 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 19 Apr 2021 01:21:46 +0000 (13:21 +1200)
commit56c11110b9067b30496d4ccb7bd3358d2758f607
treecb6b8ac839bad5c888549619ffc3521308e625a6tree | snapshot (zip tar.gz)
parentfaa0e6acb3e4378ae47bb7d97d9ca012f1d80d30commit | diff
dev/core#2493 Stop attempting to format money in the processor class

We've said this is wrong before but it has survived until now because it seemed scarier to change it. However,
now the reverse seems true.

The value in amount should always be machine friendly and there are no known processors
that expect locale specific formatting.

On the other hand the format() function is intended to prepare money for DISPLAY which
is not what is going on here
CRM/Core/Payment.php diff | blob | blame | history