Fix documented settings.php include path

[jcandan]

Problem

The relative include path is wrong.

README currently states:

Add Playwright to Drupal's Settings

Add the following line to web/sites/default/settings.php:

include '../test/playwright/node_modules/@lullabot/playwright-drupal/settings/settings.playwright.php';

The include '../test/playwright/...'; — from web/sites/default/settings.php, lands you at web/sites/, so it’s wrong.

Proposed resolution

Update instruction to use $app_root to build an absolute path:

include $app_root . '/../test/playwright/node_modules/@lullabot/playwright-drupal/settings/settings.playwright.php';

[jcandan]

It might also be good to mention that it should go AFTER any database configuration (e.g. below the IS_DDEV_PROJECT include conditional). Though, I should've already known that. 😆 😆 😆 😆

[deviantintegral]

Hm, I'm not sure this is wrong. I just checked a live project and it's using the following:

  // Include playwright settings.
  if (file_exists('../test/playwright/node_modules/@lullabot/playwright-drupal/settings/settings.playwright.php')) {
    include '../test/playwright/node_modules/@lullabot/playwright-drupal/settings/settings.playwright.php';
  }

I think normally the CWD is set to the location of index.php. Any chance your project is doing something else that would affect that?

[jcandan]

I haven't had a second to verify this to be the culprit, but I was working on a multisite. May be worth investigating, but I have to move on.

Hopefully this helps others who are seeing their local DDEV instance is being overwritten as the Umami Demo site.

[deviantintegral]

My example was also a multisite!