API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21DrupalKer...

> If a file named sites.php is present in the sites directory, it will be loaded prior to scanning for directories.

This used to be the case on 7.x, but that's wrong now.

    // Determine whether multi-site functionality is enabled.
    if (!file_exists($app_root . '/sites/sites.php')) {
      return 'sites/default';
    }

If a file named sites.php is NOT present, then directories are not scanned for at all!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

bdlangton’s picture

Status: Active » Needs review
FileSize
1.54 KB

Here is a patch that updates the documentation for DrupalKernel::findSitePath().

joachim’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks!

alexpott’s picture

Crediting @joachim for creating this issue.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed c98f167c5c to 8.6.x and 1c73bf45bc to 8.5.x. Thanks!

Backporting to 8.5.x since it is a docs fix.

  • alexpott committed c98f167 on 8.6.x
    Issue #2905111 by bdlangton, joachim: error in docs for DrupalKernel::...

  • alexpott committed 1c73bf4 on 8.5.x
    Issue #2905111 by bdlangton, joachim: error in docs for DrupalKernel::...

Status: Fixed » Closed (fixed)

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

roderik’s picture