Simple uninstall hook to remove the db table if the module is uninstalled. Opinions? Reckon losing the settings is too harsh? Do we need some kind of option offering to keep or remove the settings at uninstall time?

Questions questions.

CommentFileSizeAuthor
role_weights_uninstallhook.patch491 bytespfaocle

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
commented

+1

People have the option of disabling the module without uninstalling it. You have to explicitly go to the uninstall page. So this is correct, it should get rid of the data, if a user is taking that step.

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Assigned: Unassigned » pfaocle
Category: feature » task
Status: Active » Reviewed & tested by the community

Thanks for that.

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)