The module should remove its own variables when it is uninstalled.

CommentFileSizeAuthor
#4 backup_migrate_dropbox-2818965-hook-uninstall-variables-4.patch441 bytesgedur
#2 backup_migrate_dropbox-2818965-hook-uninstall-variables.patch439 bytesgedur

Comments

GeduR created an issue. See original summary.

gedur’s picture

gedur commented
Status: Active » Needs review
StatusFileSize
new backup_migrate_dropbox-2818965-hook-uninstall-variables.patch439 bytes

Attaching patch, please review!

tunic’s picture

tunic
he/him
Primary language Spanish
Location Madrid
commented
Status: Needs review » Needs work

The line feed at the end of the file is missing.

gedur’s picture

gedur commented
Status: Needs work » Needs review
StatusFileSize
new backup_migrate_dropbox-2818965-hook-uninstall-variables-4.patch441 bytes

Blank line added, thanks!

tunic’s picture

tunic
he/him
Primary language Spanish
Location Madrid
commented
Status: Needs review » Reviewed & tested by the community

That's a quick answer :)

  • fietserwin committed fadedd1 on 7.x-2.x authored by GeduR 
    [#2818965]: Remove module variables on hook_uninstall
fietserwin’s picture

fietserwin
Location Le Mont-Dore
commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks for reporting and supplying a patch.

We are working towards an official fully working 7.x-2.0 release. Try its alpha or dev (soon to appear) version!

Status: Fixed » Closed (fixed)

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