Based on the current HEAD install, og.module leaves a bunch of stuff in your {variables} table if you uninstall it. Attached patch fixes this and makes og_uninstall() more readable.

CommentFileSizeAuthor
og_uninstall_incomplete.patch1.58 KBdww

Comments

ajayg’s picture

ajayg commented

I suggest you have the variables alphabetically sorted so it is easier to spot a missing variable in future and update the code.

Just thought since you have touched the code anyway, it may be easier to do this one time with future variables deleting self sorted by who ever make change.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed. thanks.

Anonymous’s picture

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

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