There are two issues with node feed. 1) $node->build_mode = BUILD_MODE_RSS should be $item->BUILD_MODE_RSS. 2) The link in the rss items cannot be overriden via nodeapi op = 'rss item'.
see attached patch.

CommentFileSizeAuthor
#4 mw.patch1.3 KBmoshe weitzman
node_feed_link.patch1.06 KBdopry

Comments

birdmanx35’s picture

birdmanx35 commented

This patch applies cleanly to core.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

i tested as well as code review.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Needs work

Use of $link later was not properly modified. I am not entirely sure though that tacking the link key there is a good idea. Although I see you'd seek a way to modify it.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new mw.patch1.3 KB

fixed later use of $link. please elaborate on "not a good idea" if code changes are needed.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Well, all right, committed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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