Manual Review
You should look into hook_uninstall() to clear all the system v...

Manual Review

• You should look into hook_uninstall() to clear all the system variables you are creating. Otherwise if this module is uninstalled, it will leave behind redundant variables in the system table.

Apart from this there is not much I can see wrong in the module. Once the above is rectified this can be set to RTBC.

Cheer,
Swarad