Page MenuHomePhabricator

[WMDE-Fundraising] Allow expired utoken in Paypal donation
Closed, ResolvedPublic2 Estimated Story Points

Description

When a Paypal donation is paid by direct debit, the success notification can take up to 7 days. With the current code in DoctrineDonationAuthorizer::canUpdateDonation the update would fail because the update token has expired. Proposed solution: add the parameter $ignoreExpiredToken to canUpdateDonation

Event Timeline

@JeroenDeDauw Any other suggestions instead of using a parameter? Maybe another method canUpdateDonationwithExpiredToken?

How about renaming canModifyDonation to userCanModifyDonation and adding a systemCanModifyDonation with the same signature?

I like that idea. Much cleaner than parameters.

systemCanModifyDonation would then only check if the token matches and not if the token is expired.

Tobi_WMDE_SW claimed this task.