autosave_hidden and autosave_period (edit: and more) remain in the system.

Solutions:
a) add variable module dependency and implementation
b) add proper uninstallation

CommentFileSizeAuthor
#9 autosave-2624650-9-uninstall.patch445 bytesczigor
#5 autosave-uninstall-leaves-orphan-variables-2624650-5-7.x.patch465 bytesmimran
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

doitDave created an issue. See original summary.

doitDave’s picture

doitDave CreditAttribution: doitDave commented
Issue summary: View changes
doitDave’s picture

doitDave CreditAttribution: doitDave commented
Issue summary: View changes
mimran’s picture

mimran CreditAttribution: mimran commented
Assigned: Unassigned » mimran
mimran’s picture

mimran CreditAttribution: mimran commented
FileSize
autosave-uninstall-leaves-orphan-variables-2624650-5-7.x.patch465 bytes

Added a patch to remove the orphan variables

mimran’s picture

mimran CreditAttribution: mimran commented
Status: Active » Needs review
mimran’s picture

mimran CreditAttribution: mimran commented
Assigned: mimran » Unassigned
mimran’s picture

mimran CreditAttribution: mimran as a volunteer commented
czigor’s picture

czigor CreditAttribution: czigor at Liip for FREITAG lab. AG commented
FileSize
autosave-2624650-9-uninstall.patch445 bytes

Thanks for the report and the patch!

I do not think a https://www.drupal.org/project/variable dependency is needed, it's just adding some additional features to variables.

I would avoid using db_delete() for deleting variables. I updated the patch to use variable_del().

czigor’s picture

czigor CreditAttribution: czigor at Liip for FREITAG lab. AG commented
Status: Needs review » Fixed

  • czigor committed ad83ccc on 7.x-2.x 
    Issue #2624650 by mimran, czigor: Uninstall leaves orphan variables

Status: Fixed » Closed (fixed)

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