Hi

When I unisntalling the module, I got the error message to stop me finish.

Error: Call to undefined function xmlsitemap_clear_directory() in xmlsitemap_uninstall() (line 372 of /home/abccom/public_html/sites/all/modules/xmlsitemap/xmlsitemap.install).

CommentFileSizeAuthor
#9 xmlsitemap-improve-uninstall.patch675 bytesjacob.embree
#3 After patch.png38.4 KBriddhi.addweb
#3 Before patch.png63.89 KBriddhi.addweb
#2 xmlsitemap-3020270-2.patch477 bytesjigish.addweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

freelylw created an issue. See original summary.

jigish.addweb’s picture

jigish.addweb CreditAttribution: jigish.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Active » Needs review
FileSize
xmlsitemap-3020270-2.patch477 bytes

@freelylw, Here's the patch to resolve the error that occurs on module uninstall. Hope this helps you!. Thanks

riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
Before patch.png63.89 KB
After patch.png38.4 KB

@jigish.addweb, Thanks for providing a patch, I checked it using simplytest.me & got the same error without a patch, using your patch I found that issue is resolved. PFA

Thanks for all your work!...

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
CreditAttribution: voleger as a volunteer and at GOLEMS GABB, Drupal Ukraine Community for GOLEMS GABB, Drupal Ukraine Community commented

Looks good! Thanks

alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim at GOLEMS GABB for GOLEMS GABB commented

+1

pifagor’s picture

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

  • pifagor committed b60d230 on 7.x-2.x authored by jigish.addweb 
    Issue #3020270 by jigish.addweb, riddhi.addweb, freelylw, alex_optim,...
pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor at Drupal Ukraine Community, GOLEMS GABB for Drupal Ukraine Community, GOLEMS GABB commented
Status: Reviewed & tested by the community » Fixed
jacob.embree’s picture

jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commented
FileSize
xmlsitemap-improve-uninstall.patch675 bytes

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?

Status: Fixed » Closed (fixed)

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

ciss’s picture

ciss CreditAttribution: ciss at yousign GmbH commented

I have to agree with @jacob.embree, the committed changes only patch the symptoms, not the cause. The patch in #9 would be in line with the original intent.

@jacob.embree If this issue doesn't get reopened, do you feel like creating a follow-up?

jacob.embree’s picture

jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commented

Sure, ciss. Here it is: #3086542: Clean up uninstall routine.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor at Drupal Ukraine Community, GOLEMS GABB for Drupal Ukraine Community, GOLEMS GABB commented
Status: Closed (fixed) » Needs work
pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor at EPAM Systems, Drupal Ukraine Community, GOLEMS GABB for Drupal Ukraine Community, GOLEMS GABB, EPAM Systems commented
Status: Needs work » Fixed

This will close another task - https://www.drupal.org/project/xmlsitemap/issues/3086542

Status: Fixed » Closed (fixed)

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