The attached patch allows CCK fields to appear properly in Views RSS feeds. Without it, they suffer from the same bug that core's RSS feeds do: only the initial $node->body and $node->teaser variables are spit out, and fields turn invisible.

CommentFileSizeAuthor
views-rss-fix.patch1.57 KBeaton

Comments

WorldFallz’s picture

WorldFallz commented

Jan 1st? And I thought I was bad drupaling 12/26, lol.

Anyway, patch applies (though there seems to be some svn cruft at the top) and seems to work correctly with a quick test on a simple 1 field feed. I'll try making some more complicated feeds when I get a chance. Using d6.8 & views 6.x-2.2.

I've been trying to figure this one out for a while-- this is awesome, thanks.

and Happy New Year ;-)

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Committed! Thanks!

Status: Fixed » Closed (fixed)

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

drewish’s picture

drewish commented
Status: Closed (fixed) » Active

i really think this was the wrong way to do this, it breaks compatibility with core's rss item building... the correct fix would be for CCK to implement hook_nodeapi($op='rss item') as in #118833: Fields doesn't appear in RSS feed.

update: marked #363795: drupal_render() is not being called for RSS feeds as a duplicate.

merlinofchaos’s picture

merlinofchaos commented

More fun with RSS!

eaton’s picture

eaton commented
Status: Active » Closed (fixed)

After discussion in IRC and clarification of the way that the NODE_BUILD_* modes operate, the problem really seems to be a core bug that needs dealing with... #365183: node_feed() silently discards $node->content has patches for D6 and D7...