Tried to run update.php on the new version and got:

Parse error: syntax error, unexpected ')' in /home/monarchd/public_html/sites/all/modules/xmlsitemap/xmlsitemap.install on line 161

Anyone else see this?

CommentFileSizeAuthor
#2 xmlsitemap.patch326 bytesjustin.hopkins

Comments

amardeshbd’s picture

Title: update.php for 6.x-1.x-dev » Error on Update: update.php for 6.x-1.x-dev

Yep !! I've got this too ... when updating from 0.x to 1.x
Parse error: syntax error, unexpected ')' in /home/uwindsor/public_html/hossainkhan.info/sites/all/modules/xmlsitemap/xmlsitemap.install on line 161

justin.hopkins’s picture

StatusFileSize
new326 bytes

Removing the comma at the end of line 161 and placing a semi colon after the closing parens on line 162 should fix you up.

Afterthought: Here's a patch.

Disclaimer: Try at your own risk. update.php threw some warnings - not able to unlink a .svn folder(fine, but probably should be taken into consideration by module developer), unable to drop some tables. No big deal, the module seems to work fine now. Anyway, this isn't for production sites ;)

To apply the patch:
Put the patch file into the xmlsitemap module directory and:
patch < xmlsitemap.patch

Ryanbach’s picture

Status: Active » Reviewed & tested by the community
avpaderno’s picture

Title: Error on Update: update.php for 6.x-1.x-dev » Parse error
Status: Reviewed & tested by the community » Fixed

The parse error was already fixed in CVS.
I changed the code to remove the temporary files so that it suppress warnings; it also try to delete any sub-directory it finds (up to four levels). The directory .svn was not created by the module, which creates only files into its own temporary directory.

Setting this report as duplicate of #338907: Parse error.

avpaderno’s picture

Status: Fixed » Closed (duplicate)