- Issue created by @phil stringer
- 🇺🇸United States hyperlogos
try out this patch I whipped up against 2.0.0, I lifted the solution essentially verbatim from https://www.drupal.org/node/3000490 →
- 🇬🇧United Kingdom phil stringer
That stops it breaking drupal 10. Thanks.
- 🇺🇬Uganda Thomas Kaisuka
After applying that patch, i still get this error in Drupal 10.0.7 with Commerce 2.0 installed
- Status changed to Needs review
over 1 year ago 10:20am 1 April 2023 -
andileco →
committed 2b1ae886 on 8.x-1.x authored by
hyperlogos →
Issue #3337606 by hyperlogos, Thomas Kaisuka, Phil Stringer, apaderno,...
-
andileco →
committed 2b1ae886 on 8.x-1.x authored by
hyperlogos →
-
andileco →
committed 2b1ae886 on 2.0.x authored by
hyperlogos →
Issue #3337606 by hyperlogos, Thomas Kaisuka, Phil Stringer, apaderno,...
-
andileco →
committed 2b1ae886 on 2.0.x authored by
hyperlogos →
- 🇺🇸United States andileco
Thank you for this ticket! Fixed and issued a new release.
- Status changed to Fixed
over 1 year ago 4:49pm 17 April 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- 🇵🇱Poland sebaz
It is not fixed because module still remains as compatible with all 9.x versions, it should require
^9.4 | 10
or split into 2 versions like Reroute Email done it earlier 🐛 Use of ModuleHandler::invokeAllWith() breaks Drupal with core < D9.4 Fixed :- one branch for older ^8.8 || <9.4
- second branch for ^9.4 || ^10