- Issue created by @zaporylie
- Merge request !296Issue #3466085 by zaporylie: Utilize template method pattern for ensuring... → (Open) created by Unnamed author
- 🇮🇱Israel jsacksick
1. all contrib payment gateways will likely have to adapt to the new strategy but it hasn't been documented in any way
This is a common pattern, used everywhere in Drupal core... And also, we can fix that and update the documentation and/or create a change record. As a reminder, Commerce 3 hasn't been released yet...
Also, this change would break Braintree & Stripe for example, since they're implementing their own init() method with a different signature (the container isn't passed).
Still unsure I wanna proceed with this... I know at the very least Braintree, Paypal, Stripe are already ready and work with the current 3.x dev, and we can update the others if necessary.
- 🇮🇱Israel jsacksick
FYI, I've created a change record: https://www.drupal.org/node/3468511 →
- Status changed to Closed: works as designed
about 1 month ago 12:30pm 16 August 2024 - 🇳🇴Norway zaporylie
I've been thinking about this for a while, and it grew on me. So I am ok just closing this with a CR in place.