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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jacob.embree created an issue. See original summary.

jacob.embree’s picture

jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commented
Issue summary: View changes
alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim at GOLEMS GABB, Drupal Ukraine Community for GOLEMS GABB, Drupal Ukraine Community commented

Looks good.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at EPAM Systems, Drupal Ukraine Community for Drupal Ukraine Community, EPAM Systems 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
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at EPAM Systems, Drupal Ukraine Community for Drupal Ukraine Community, EPAM Systems commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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