I was getting an error when I tested with amounts above 1000. So I did some debugging and I found out that the amount is formatted with number_format.

https://git.drupalcode.org/project/mollie_payment/blob/7.x-2.x/includes/...

The default thousands separator of the function is a comma. The Mollie API expects no separator for thousands.

So my fix is to change the number_format function to get the format the Mollie API expects.

CommentFileSizeAuthor
#2 mollie-payment-patch-3052203-1.patch1.23 KBThomasDik
mollie_payment.patch826 bytesdavid.duymelinck_mia.be
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

david.duymelinck_mia.be created an issue. See original summary.

ThomasDik’s picture

ThomasDik CreditAttribution: ThomasDik at Synetic commented
FileSize
mollie-payment-patch-3052203-1.patch1.23 KB

Added a small change to this patch as there were two places where this type of number formatting was performed.

sjerdo’s picture

sjerdo
Primary language Dutch
Location Haarlem
CreditAttribution: sjerdo at Synetic commented
Status: Active » Reviewed & tested by the community

Looks good to me!

  • ricovandevin committed 0bc8e98 on 7.x-2.x 
    Issue #3052203 by david.duymelinck_mia.be, ThomasDik: Mollie API REST...
ricovandevin’s picture

ricovandevin CreditAttribution: ricovandevin at Finlet for Carwebs commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.