Feeds should not mark submodules as `required`, as this hides the module from the module UI.

[argiepiano]

Feeds should not mark submodules as `required`, as this hides the module from the module UI.

[argiepiano]

PR #142 ready for review.

[argiepiano]

Again a test failure - no time to check, but this may be unrelated.

[MegaChriz]

This "required" mark was added to avoid modules accidentally getting disabled while they were still in use by importers. I'm not familiar with Backdrop yet, but the idea is that you are not allowed to disable the module in question (via the UI or on the command line [for example: using drush, not sure if drush is compatible with backdrop]) until all importers that depend on it are removed. If the "disabled" mark still preserves this behavior, then that's great!

See here the issue in which the "required" mark was added: https://www.drupal.org/node/2379407

At first glance, the test failures look unrelated. I saw some messages about classes that could not be found. Based on your changes, I would find it highly unlikely that that would result in tests not be able to find classes.

[argiepiano]

Thanks, @MegaChriz!! Yes, in BAckdrop, the key disabled disables the checkbox and prevents disabling the module. It's used by the field core module to avoid disabling it when there are fields. See this line

[herbdool]

Merged