API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Dependenc...
Enter a descriptive title (above) relating to interface ContainerInjectionInterface, then describe the problem you have found:
The docs for this interface ContainerInjectionInterface look like they were just left over from when it maybe was named ControllerInterface or copy/pasted from there. They need to be updated to reflect what the interface actually does/is/should be used for.
As this documentation is pretty much unrelated to the class (assuming that the name is a sensible choice), it's a major bug, especially since every route controller class is supposed to extend this interface so it's fairly prominent.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2105693-3.patch | 2.17 KB | cilefen |
Comments
Comment #1
tim.plunkett#2165475: Provide a generic class resolver better codifies where this is used, and writing these docs might be easier once that's in. Not blocked by any means though.
Comment #2
dawehnerRemove pointless tag.
Comment #3
cilefen commentedComment #4
jhodgdonLooks reasonable to me, thanks!
Comment #5
cilefen commentedI am glad I have hacked away at core enough now to have seen this pattern a lot.
Comment #6
alexpottThis issue addresses a major bug and is allowed per https://www.drupal.org/core/beta-changes. Committed 957078e and pushed to 8.0.x. Thanks!