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.
The parseDependency method does not belong into the ModuleHandler. It's a general utility thing. And while we are at it we can just as well move the checkIncompatibility method over as well.
Comment | File | Size | Author |
---|---|---|---|
#7 | drupal8.base-system.2098521-7.patch | 11.93 KB | fubhy |
#1 | drupal8.base-system.2098521-1.patch | 7.85 KB | fubhy |
Comments
Comment #1
fubhy CreditAttribution: fubhy commentedComment #3
fubhy CreditAttribution: fubhy commentedWe already got more than enough crap in ModuleHandler so I really want to get rid of this additional burden. I am going to do some further cleanup in a couple of other issues and this would really help to keep that class clean.
Comment #4
fubhy CreditAttribution: fubhy commented#1: drupal8.base-system.2098521-1.patch queued for re-testing.
Comment #6
tstoecklerYes, I agree with this issue. The patch is missing the actual file.
Comment #7
fubhy CreditAttribution: fubhy commented/facepalm
Comment #8
dawehnerIs there a reason why this is not part of the extension namespace at least?
Comment #9
dawehner.
Comment #10
dawehnerThis is now part of the module handler.