This is a followup to #3020270: error when uninstall comment #9.

The only reason xmlsitemap_clear_directory() might not exist is if xmlsitemap.module hasn't been loaded. The very next line loads xmlsitemap.module, so why not load xmlsitemap.module first, then run xmlsitemap_clear_directory(), then remove variables?

The patch from that comment is attached here.

CommentFileSizeAuthor
xmlsitemap-improve-uninstall.patch675 bytesjacob.embree

Comments

jacob.embree created an issue. See original summary.

jacob.embree’s picture

jacob.embree commented
Issue summary: View changes
alex_optim’s picture

alex_optim
Location Tallinn
commented

Looks good.

pifagor’s picture

pifagor commented
Status: Needs review » Reviewed & tested by the community

  • pifagor committed 8addf71 on 7.x-2.x authored by jacob.embree 
    Issue #3086542 by jacob.embree, alex_optim, pifagor: Clean up uninstall...
pifagor’s picture

pifagor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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