We've been getting a little slack about this.

This module sets a bazillion variables, manipulates database tables, etc. We should be good Drupal citizens and clean up after ourselves when we disable the module.

Comments

michelle’s picture

Disable? Or uninstall?

Michelle

webchick’s picture

Well. Eventually should be uninstall. But while we're in development it's much easier to just check/uncheck a box than check/uncheck a box, go to a tab, check another box, click yes on a confirmation button, etc. :)

michelle’s picture

Ok, just clarifying. Makes sense in dev but won't want that in the final version. :) I can hear the users now... "I disabled drubb and all my settings changed!"

Michelle

merlinofchaos’s picture

Yeah and devel's "reinstall module" button is even easier. =)

webchick’s picture

Title: Clean up all of the crap in drubb_disable() » Clean up all of the crap in drubb_uninstall()

Well, touché. ;)