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.
Problem/Motivation
DefaultConfigTest
has some leftover @todo
s pointing to #2208429: Extension System, Part III: ExtensionList, ModuleExtensionList and ProfileExtensionList and #2186491: [meta] D8 Extension System: Discovery/Listing/Info.
The first issue is now fixed, and it seems that we can remove both workarounds.
Proposed resolution
Remove the dead code.
Remaining tasks
Review the patch.
User interface changes
Nope.
API changes
Nope.
Data model changes
Nope.
Comment | File | Size | Author |
---|---|---|---|
#5 | interdiff-5.txt | 1.04 KB | amateescu |
#5 | 3014011-5.patch | 2.03 KB | amateescu |
#2 | 3014011.patch | 1023 bytes | amateescu |
Comments
Comment #2
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedSpotted this while working on some other unrelated things :)
Comment #3
andypostGot spot but there's more
Comment #4
BramDriesenSetting to needs work for #3
Comment #5
amateescu CreditAttribution: amateescu for Pfizer, Inc. commented@andypost, good point :) Removed the same workarounds from
ModuleHandlerTest
.The
ThemeHandler
part is being done in #2659940: Extension System, Part III: ThemeExtensionList and ThemeEngineExtensionList, as forModuleInstaller
I couldn't find the exact issue where that's supposed to be cleaned up, but it's certainly not in the scope of this one.Comment #6
BramDriesenI think this captures everything of the scope of this issue :) code removal looks good.
Comment #7
catchCommitted a11f188 and pushed to 8.7.x. Thanks!