Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Updated: Comment #0
Problem/Motivation
The HookDiscovery class (allowing plugins to be discovered with hooks) does not have enough (any) documentation:
/**
* Provides a hook-based plugin discovery class.
*/
class HookDiscovery implements DiscoveryInterface {Proposed resolution
Write proper docs
Remaining tasks
User interface changes
N/A
API changes
N/A
Related Issues
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | hook-2100249-1.patch | 528 bytes | tim.plunkett |
Comments
Comment #1
tim.plunkettNot sure what else to say.
Comment #2
xjmMoving to the documentation queue for @jhodgdon.
Comment #3
jhodgdonOK. I'm not sure this documentation really adds much, since the constructor does provide this information and so does the $hook variable... Maybe you could explain what the hook would need to do or point to an example or something?
And could you also replace the outdated "Implements ..." docs on the methods with @inheritdoc? Please? Just two methods...
Comment #15
quietone CreditAttribution: quietone at PreviousNext commentedI agree that more documentation is needed for this class, however the existing documentation is neither misleading nor incorrect, therefor changing to a task.