Especially In multi-language environments, it would be quite useful if the entries in the feed would contain the node language as xml:lang attribute value

Comments

mgifford’s picture

Version: 6.x-1.1 » 7.x-1.x-dev
Issue tags: +i18n

I'm bumping this to D7 as hopefully it will be adopted there and then back-ported to D6.

deekayen’s picture

Status: Active » Needs work

There has been a lot of work to the module since this patch was written. Please re-roll for 7.x.

deekayen’s picture

When I looked at my personal node table, the language field is set to "und". I don't know how that translates to the $node variable on node_load(), nor do I have much interest in making language support on my own site. Seems to me, this patch is incomplete and needs some additional variable setting to $item in _atom_print_feed() to be able to pass the language to the theme function.

bart.hanssens’s picture

Probably changed in D7, it's been a while since I've added the patch, but I'll take a look at it.

sinkinsona’s picture

Any update on the xml:lang patch? Just wanted to see if any progress has been made as I have a requirement for a project that I am working on.

bart.hanssens’s picture

Status: Needs work » Needs review
StatusFileSize
new1.07 KB

Maybe this patch will help.

deekayen’s picture

Status: Needs review » Fixed

committed

Status: Fixed » Closed (fixed)
Issue tags: -i18n

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