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

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Issue tags: +multi-site
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

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

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

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

catch
he/him
Primary language English
commented
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

(not verified) commented
Issue summary: View changes

Updated issue summary.