It should be possible for the xmlsitemap_i18n.module to provide a 'Language neutral' sitemap.

CommentFileSizeAuthor
#5 787760-xmlsitemap-context-catch-all-D7.patch16.56 KBdave reid
#2 787760-xmlsitemap-context-catch-all-D7.patch5.25 KBdave reid
#1 787760-xmlsitemap-context-catch-all-D6.patch4.89 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 787760-xmlsitemap-context-catch-all-D6.patch4.89 KB

I think something like this is all it needs...

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 787760-xmlsitemap-context-catch-all-D7.patch5.25 KB

Rolled for D7 as well.

Status: Needs review » Needs work

The last submitted patch, 787760-xmlsitemap-context-catch-all-D7.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review

#2: 787760-xmlsitemap-context-catch-all-D7.patch queued for re-testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 787760-xmlsitemap-context-catch-all-D7.patch16.56 KB

Playing around with a new xmlsitemap_test.module found a few bugs with this implementation.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Priority: Normal » Major
chris matthews’s picture

chris matthews commented
Assigned: dave reid » Unassigned
Issue summary: View changes
Priority: Major » Normal
Status: Needs review » Needs work
Issue tags: +Needs reroll

The 9 year old patch in #5 of course does not apply to the latest xmlsitemap 7.x-2.x-dev and if still applicable needs a reroll.