In the xmlsitemap_output function you must put the '$link_count = _xmlsitemap_link_count();' line before 'if (isset($chunk)) {' so that $link_count is available for the 'if ($link_count > $chunk_size) {' line.

Sorry but i cannot make a patch right now.

Comments

andreiashu’s picture

andreiashu commented
Title: Bug in xmlsitemap_output function » xmlsitemap_output_index never gets called
Priority: Normal » Critical
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: xmlsitemap_output_index never gets called » xmlsitemap_output_index() is never called
Status: Active » Postponed

Thanks for your report. I will correct the problem.

There is no problem with not having a patch; the explanation you gave is enough to understand what is wrong in the code, and you also gave the solution to the problem. That is already a lot of work, and it's well appreciated.

avpaderno’s picture

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

It has been fixed in CVS.
Thanks again for your report.

Status: Fixed » Closed (fixed)

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

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: xmlsitemap_output_index() is never called » $links_count is not initialized