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:

Every single test module is visible on the modules page

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.)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

As 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.

webchick’s picture

Status: Needs review » Reviewed & tested by the community

Yeah, 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.

webchick’s picture

Title: Default 'extension_discovery_scan_tests' to FALSE in example.settings.local.php » Don't show 80 bazillion test modules on the modules page when in development mode
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

I 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!

  • alexpott committed 829715b on 8.0.x
    Issue #2342435 by tstoeckler, webchick: Don't show 80 bazillion test...

Status: Fixed » Closed (fixed)

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