Skip to content

Always try to update a credit card

jamie requested to merge jamie/stripe:5.x_noemail_cc_take2 into 5.x_noemail

Otherwise we may ignore a credit card provided by a user and instead charge the one associated with their account.

https://lab.civicrm.org/extensions/stripe/issues/4

Without this additional patch, a user can submit a membership contribution page but only include a membership amount and not include an additional donation.

The membership amount is considered a secondary financial transaction, so it won't trigger an update to their customer record with the credit card info they are submitting.

As a result, whatever credit card is on file for them will get submitted instead.

With this patch, we always try to update the customer with the credit card submitted no matter what. And if we get an error, it means it was already updated so we can simply ignore it.

Merge request reports