- Issue created by @maxilein
- 🇦🇹Austria maxilein
drush cr runs fine and gives no error.
I've have increased the php memory_limit to 2048M just to make sure. - 🇦🇹Austria maxilein
It is also happening with 4.1.x dev than with 4.0.1
But the loading of pages gets a lot further... - 🇦🇹Austria maxilein
I had to go back to D10.2.7:
then uninstall
the module Content Dependency Tracker and Core Event Dispatcher which was why I needed hook_event_dispatcher in the first place.Then the upgrade to D10.3 worked. The error is gone.
- 🇧🇪Belgium jelle_s Antwerp, Belgium
After some digging around, I found 📌 Replace RequestCloseSubscriber with needs_destruction tag on ModuleHandler Fixed , where Drupal core's ModuleHandler got the
needs_destruction
service tag. It seems that sinceHookEventDispatcherModuleHandler
decorates that service, it also has theneeds_destruction
tag, meaning that it should probably also implementDrupal\Core\DestructableInterface
and thedestruct
method. That interface has existed since Drupal 8.0.0, so it's safe to implement without having any BC breaks. - 🇧🇪Belgium mauro_
I agree with Jelle_S and I think the solution of #10 seems adequate. Thanks!
Here's a patch for anyone who needs to upgrade quickly:
- Status changed to Needs review
5 months ago 12:06pm 21 June 2024 -
el7cosmos →
committed 9ac7ec18 on 4.x authored by
Jelle_S →
Issue #3456138 by Jelle_S, Mauro_, maxilein, el7cosmos: Call to...
-
el7cosmos →
committed 9ac7ec18 on 4.x authored by
Jelle_S →
- Status changed to Fixed
5 months ago 12:57pm 21 June 2024 -
el7cosmos →
committed a89a84bf on 4.1.x authored by
Jelle_S →
Issue #3456138 by Jelle_S, Mauro_, maxilein, el7cosmos: Call to...
-
el7cosmos →
committed a89a84bf on 4.1.x authored by
Jelle_S →
Automatically closed - issue fixed for 2 weeks with no activity.