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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
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
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
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
CreditAttribution: Dave Reid 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
CreditAttribution: Dave Reid commented
FileSize
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
CreditAttribution: Dave Reid commented
Priority: Normal » Major
Chris Matthews’s picture

Chris Matthews CreditAttribution: 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.