- Issue created by @donquixote
- Merge request !12322Draft: Add ModuleHandlerTest::testAddModuleDestroysHookImplementations(). β (Open) created by donquixote
- πΊπΈUnited States nicxvan
I really don't think people are using this.
It was a helper for Drupal install and updating and wasn't even working there for ages.
We can track it though.
Unless someone pops up though with a valid use case this is already deprecated. I'm not sure it's worth the effort to change.
- πΊπΈUnited States nicxvan
Inn fact I'm tempted to mark this postponed maintainer needs more info on an example of someone actually using this.
- π©πͺGermany donquixote
My main concern here is whenever we refactor this, e.g. in π [pp-1] Explore adding an ImplementationList object for hook implmentations Active , we are replicating severely broken functionality.
- πΊπΈUnited States nicxvan
Just to clarify you mean replicating broken functionality only within the add method right?
- π©πͺGermany donquixote
It was a helper for Drupal install and updating and wasn't even working there for ages.
It was working before the OOP hooks via EventDispatcher. So in 11.0.
Unless someone pops up though with a valid use case this is already deprecated. I'm not sure it's worth the effort to change.
Normally "deprecated" code still works as advertised or as expected, you should stop using it to prepare for the next major version.
- πΊπΈUnited States nicxvan
Postponing until we have someone impacted by this.
Contrib projects that use this method would be most helpful, but I've not been able to find any contrib using it.
Feel free to set back to active if you find contrib using it and we can take another look.
- πΊπΈUnited States nicxvan
I searched http://codcontrib.hank.vps-private.net/search?text=Drupal%3A%3Aservice%2...
And the other common ways this could be called and all returned 0 usages.