The error is caused by the function name used. It has been fixed in CVS.

CommentFileSizeAuthor
#4 456288-xmlsitemap-node-get-priority-D6-1.patch952 bytesdave reid

Comments

epimeth’s picture

epimeth commented

what should be the correct function name?

epimeth’s picture

epimeth commented

I changed it to xmlsitemap_node_get_priority... now I don't crash, but I also don't have any nodes in my sitemap... I enabled the module, cleared the sitemap cache, cleared the site cache, and went to sitemap.xml.... all I have there is the homepage!

epimeth’s picture

epimeth commented

and I tried the same exact thing again and it *did* work... great :-)

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Fixed » Needs review
StatusFileSize
new 456288-xmlsitemap-node-get-priority-D6-1.patch952 bytes

There is also an error in the xmlsitemap_node_get_priority() function. It has a parameter $load, but in the code of the function, it is never used. Instead there is a variable $reload (which I assume should be $load) that will be undefined. Patch attached for review.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Fixed

It seems I have some problems with the editor; for both the files where the error is present, the editor gave me a warning about the files not being correctly saved.

Anyway, I re-changed the files, and committed the change in CVS.

Status: Fixed » Closed (fixed)

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