Problem/Motivation
#2226761: Change all default settings and config to fast/safe production values was great, but I think it got one thing wrong: the location of example.settings.local.php.
Visually it sits nicely along with example.sites.php, but it's prone to errors and "why is my settings.local.php not working?!" because you can't just duplicate it into the same directory and rename it. Because the settings.local.php include on the bottom of settings.php is conditional, it can also be painful to diagnose this misplaced file.
The location of sites/example.sites.php makes sense, because you can just duplicate it in place. sites.php lives in sites, that should stay as is. But a settings.php/settings.local.php file sitting in the sites folder does not make sense and won't get picked up.
Proposed resolution
Move example.settings.local.php from sites to sites/default and update any relevant documentation.
Remaining tasks
Discuss
Patch
Change record?
User interface changes
n/a
API changes
n/a
Data model changes
n/a
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | 2618158-move-example-settings-local-15.patch | 9.84 KB | jcnventura |
Comments
Comment #2
star-szrComment #4
corbacho commentedI agree. The convention is that example. or default. prefixed files, can be copied & renamed in the same folder. Doesn't have much sense in the current folder
Comment #15
jcnventuraI fully agree with moving this file into default. I've been working with Drupal 8 and 9 for years, and only now realized that there is an example.settings.local.php file in the wrong location. I was not even aware that this file existed before this week. It MUST be moved into the default directory.