Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
Once #2208429: Extension System, Part III: ExtensionList, ModuleExtensionList and ProfileExtensionList is in we could try starting install module like any other method.
Once this is done we can for example get rid of the ugly hack in drupal_install_system
:
\Drupal::service('extension.list.module')->setFilename('system', 'core/modules/system/system.info.yml');
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#17 | 2934063-17.patch | 1.94 KB | alexpott |
#15 | 2719315-15.patch | 841 bytes | alexpott |
Comments
Comment #8
RoSk0Come across this one when debugging #2766509: Gracefully handle a non-existent cache service like cache.backend.memcache.
Lets ask testbot is it works.
Comment #9
RoSk0Lots of tests failed...
Comment #10
amit.drupal CreditAttribution: amit.drupal as a volunteer and for Valuebound commentedUpdate path #8 I hope its working.
Comment #11
amit.drupal CreditAttribution: amit.drupal as a volunteer and for Valuebound commentedComment #15
alexpottComment #16
daffie CreditAttribution: daffie commentedIn the Drupal code base there is another reference to this issue in the method Drupal\Core\Extension\ExtensionList::reset(). That @todo should also be fixed/removed.
Comment #17
alexpottI think that's really more to do with #2934063: Remove the workaround in \Drupal\Core\Installer\InstallerModuleExtensionList::setPathname.
Let's try to do what it wants to do here - but I'm not really sure it is that wise.