Thanks for the handy module. You should add .install file which will remove all the variables defined.

Comments

Chill35’s picture

Chill35 commented

The module needs no .install file because it creates no table entry.

As far the system variables it uses, an install file will have no impact on that. To re-init these values to their 'manufacture' defaults, visit the module's config page. Otherwise, if you stop using the module, just disable it, and don't worry about a thing.

Shit, HOOK_uninstall can delete the variables entries used by a module.

Remove any tables or variables that the module sets.

http://api.drupal.org/api/function/hook_uninstall/6

Thanks.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

This is incorrect - have a look in Devel module for example implementing http://api.drupal.org/api/function/variable_del in hook_uninstall().

Chill35’s picture

Chill35 commented
Priority: Normal » Minor

Thanks, Amitaibu,

I am giving the module some love this afternoon, so I will look into this. Several months later. Never not too late to do the right thing.

Chill35’s picture

Chill35 commented
Version: 6.x-1.8 » 6.x-1.9
Assigned: Unassigned » Chill35
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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