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.
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.