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.

CommentFileSizeAuthor
next.install-conf-path.0.patch1005 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Issue tags: +multi-site
moshe weitzman’s picture

Status: Needs review » Reviewed & tested by the community

Wow, I never knew about $require_settings param. Noted for drush site-install.

webchick’s picture

When either of you has a moment, could you come over and help me understand more what this is doing and why it's needed?

catch’s picture

Status: Reviewed & tested by the community » Fixed

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

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.