pronamic / wp-pronamic-pay

The Pronamic Pay plugin allows you to easily accept payments with payment methods like credit card, iDEAL, Bancontact and Sofort through a variety of payment providers on your WordPress website.
https://pronamicpay.com
34 stars 14 forks source link

Subscription 'On Hold' by expired mandate change payment, not reactivated with later successful mandate change payment #318

Closed rvdsteege closed 2 years ago

rvdsteege commented 2 years ago

The following scenario occurred when trying to update the mandate for a subscription:

Note: customer is using Pronamic Pay version 7.0.2, I've not yet tried to reproduce this with the latest release.

My first thought is that the first expired payment should not have resulted in the subscription status being updated to 'On Hold', as mandate changes should probably not have any influence on the activation status of the subscription.

Internal Help Scout ticket: https://secure.helpscout.net/conversation/1831339824/23764/

rvdsteege commented 2 years ago

Tested and unable to reproduce.

Seems to be resolved with https://github.com/pronamic/wp-pay-core/commit/e9d28d8957ab8daff7312c385b4852eb606217c0 as $subscription->is_first_payment( $payment ) uses the payment periods.