Installing non-necessary modules during an update is quite undesirable, conflicts if ya have a hook_node_info defining 'feed' for example (array_merge_recursive BOOM), requires making an update to run after feeds to remove the modules when doing automated deployment, etc.

Patch just removes the enable, but it's a bit extreme.

CommentFileSizeAuthor
feeds_install_no_want_2.patch1.15 KBhefox

Comments

ceardach’s picture

Category: feature » bug
Priority: Normal » Major

I'm changing this to a bug report and raising the priority. Its pretty bad form for a contrib module -- and a framework module at that -- to automatically enable a module during installation or update.

dave reid’s picture

That is ... interesting.

dave reid’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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