I anticipate approximately 99.99999999% of developers wanting to use settings.local.php in D8 to help shortcut development settings.
I anticipate 0.0000000001% of them wanting their modules page to look like this:
This is particularly confusing for new users.
This patch changes the default from TRUE to FALSE but leaves the docs in so they know why they might want to toggle it back (though personally I'd just copy the test module into the /modules folder. This seems like a bit of a silly setting.)
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 2342435-1-collapse-testing-modules.patch | 714 bytes | tstoeckler |
| true-to-false.patch | 455 bytes | webchick | |
| Screen Shot 2014-09-21 at 10.26.02 PM.png | 44.29 KB | webchick |
Comments
Comment #1
tstoecklerAs I mentioned in #2327095: Document how to allow test module and theme installation in example.settings.local.php I would much prefer to just collapse that details element by default. Attached patch does that. Even though we now have that setting documented, I'm not sure how many developers will find that, especially when using
drush en.Comment #2
webchickYeah, I guess that's a fair point, though I have never personally used that method of debugging.
I prefer my patch, but this approach works as well.
Comment #3
webchickComment #4
alexpottI don't think copying the test module around is a good thing to teach new developers - since they'll have fun with git and new files and diffing any changes. Going for the patch in #1. Discoverability of example.settings.local.php is problematic for new developers let alone this specific setting. Personally I feel the installer should have a developer profile or checkbox that sets up the environment in a developer friendly way.
Committed 829715b and pushed to 8.0.x. Thanks!