I see that system_install() automatically installs comments and taxonomy. I haven't checked comments, but I have looked at taxonomy and it has no uninstall code. These functions need to be in their respective modules. And if a module is "optional" why is it enabled by default (e.g. taxonomy)?

Comments

Damien Tournoud’s picture

Indeed, system_install() is a big mess that needs to be refactored.

NancyDru’s picture

This is fixed in 7.x. I copied the install and uninstall code from 7.x into 6.x and that worked, except that the default profile keeps trying to turn on Taxonomy again. This breaks Simpletest.

rootwork’s picture

Status: Active » Closed (won't fix)

There's some explanation in this thread: #360104: Allow taxonomy to be uninstalled - move install/uninstall from system.install to taxonomy.install

Essentially: This won't be fixed in D6 because modules in D6 assume that taxonomy and comments are enabled, so backporting the fix from D7 would cause things to break.

NancyDru’s picture

IMHO, anyone that requires either of those (or color, etc) and does not have a dependency coded in .info deserves to be broken...

NancyDru’s picture

Version: 6.x-dev » 7.x-dev
Status: Closed (won't fix) » Active

This is not fixed in D7 either. There are two profiles in D7, each enables extra modules:

expert
'block', 'dblog'
defualt
'block', 'color', 'comment', 'help', 'menu', 'taxonomy', 'dblog'

Okay, I could argue dblog so that there is some kind of logging available for getting the site started (install, update, simpletest).

"Block" was moved from required to optional - let's make it optional...

stephencamilo’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)
hestenet’s picture

Status: Closed (won't fix) » Active

Reset issue status.