Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Spin-off from #1626584: Combine configuration system changes to verify they are compatible
The Drupal installer cannot be fully tested in a separate sub/multi-site.
That is, because you have to provide a settings.php already.
The dependency on settings.php is unnecessary. We're able to determine the $confdir via conf_path(), without requiring a settings.php.
Comment | File | Size | Author |
---|---|---|---|
next.install-conf-path.0.patch | 1005 bytes | sun | |
Comments
Comment #1
sunComment #2
moshe weitzman CreditAttribution: moshe weitzman commentedWow, I never knew about $require_settings param. Noted for drush site-install.
Comment #3
webchickWhen either of you has a moment, could you come over and help me understand more what this is doing and why it's needed?
Comment #4
catchSo this just means you need to know what the conf_path() is without trying to require a settings file at all - it initializes the global but doesn't do anything else.
Looks fine to me so committed/pushed to 8.x.
Comment #5.0
(not verified) CreditAttribution: commentedUpdated issue summary.