Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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).
Comment | File | Size | Author |
---|---|---|---|
#9 | xmlsitemap-improve-uninstall.patch | 675 bytes | jacob.embree |
#3 | After patch.png | 38.4 KB | riddhi.addweb |
#3 | Before patch.png | 63.89 KB | riddhi.addweb |
#2 | xmlsitemap-3020270-2.patch | 477 bytes | jigish.addweb |
|
Comments
Comment #2
jigish.addweb CreditAttribution: jigish.addweb at AddWeb Solution Pvt. Ltd. commented@freelylw, Here's the patch to resolve the error that occurs on module uninstall. Hope this helps you!. Thanks
Comment #3
riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented@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!...
Comment #4
volegerLooks good! Thanks
Comment #5
alex_optim+1
Comment #6
pifagorComment #8
pifagorComment #9
jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commentedThe 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 runxmlsitemap_clear_directory()
, then remove variables?Comment #11
ciss CreditAttribution: ciss at yousign GmbH commentedI 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?
Comment #12
jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commentedSure, ciss. Here it is: #3086542: Clean up uninstall routine.
Comment #13
pifagorComment #14
pifagorThis will close another task - https://www.drupal.org/project/xmlsitemap/issues/3086542