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.
Comment | File | Size | Author |
---|---|---|---|
#12 | tip-1967406-12.patch | 2.89 KB | tim.plunkett |
#12 | interdiff.txt | 898 bytes | tim.plunkett |
#4 | tip-1967406-4.patch | 2.88 KB | tim.plunkett |
#3 | tip-1967406-3-do-not-test.patch | 2.88 KB | tim.plunkett |
#2 | tip-1967406-2-do-not-test.patch | 3.97 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettThis incorporates the patch from #1967420: Allow Core\AnnotatedClassDiscovery to pass all parameters to the constructor of Component\AnnotatedClassDiscovery
Comment #2
tim.plunkettHere's just the changes for this issue.
Comment #3
tim.plunkettWell, why bother with a one-off discovery that is not reusable?
Comment #4
tim.plunkettThat other patch went in.
Comment #5
tim.plunkettTagging.
Comment #6
dawehnerA general point I want to raise here: This removes the possiblity to override the tour module, by putting it into /modules instead of /core/modules. I'm not sure whether this is still possible at the moment in D8, but it was certainly in previous time.
Comment #7
EclipseGc CreditAttribution: EclipseGc commentedOverriding the Manager class should still be possible, and if that's possible, then this line can be changed. No?
Eclipse
Comment #8
tim.plunkett#4: tip-1967406-4.patch queued for re-testing.
Comment #10
tim.plunkett#4: tip-1967406-4.patch queued for re-testing.
Comment #11
dawehnerOverriding the Manager class should still be possible, and if that's possible, then this line can be changed. No?
Sure but do you expect that you can't move drupal modules but you require to alter the manager? What about using basename or __DIR__ or something like that here?
Comment #12
tim.plunkettBorrowing a trick from @EclispeGC, this does exactly the same thing.
Comment #13
dawehnerThat's a great tip!!
Comment #14
alexpottCommitted a78b614 and pushed to 8.x. Thanks!