localisation issue

[catorghans]

When having standard Dutch localisation settings
with
Thousands Separator being .
and
Decimal Delimiter being ,

processing Contribution Pages and Webforms with the Mollie payment provider gives:
Invalid integer part 10,00. Invalid digit , found

This error did not occur in 3.21, but does in 3.24

[yurg]

This commit 4d14b3d#diff-b16e98bc6ceb311e4138d0e506d5a26518d797cabeec1f9f468917ea8fd20b08R592 fixes the issue in v3.24 Regression perhaps.

[eileenmcnaughton]

Have you got steps to replicate with Contribution pages? It looks like the commit @yurg points to should have fixed it but I feel you are saying it broke it?

[yurg]

@eileenmcnaughton I got a message yesterday regarding 4-digits donation fail due to "Invalid amount passed in as money - {money} Array ( [money] => 1 000.00 )" and landed here from this page https://civicrm.stackexchange.com/questions/48342/issue-events-online-payment-invalid-integer-part-100-00-invalid-digit
Have applied only this one 4d14b3d#diff-b16e98bc6ceb311e4138d0e506d5a26518d797cabeec1f9f468917ea8fd20b08R592 from all mentioned on SE and it seems to be working - still waiting for confirmation from reporters.

[eileenmcnaughton]

@yurg right -but I'm confused - cos that patch is already merged?

oh - not tagged in a release huh

3.24...master

[eileenmcnaughton]

OK - looks like it was all done for 3.25 but not tagged so I tagged it https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/releases/tag/3.25

[yurg]

@eileenmcnaughton Thank you!