Closed (fixed)
Project:
Views (for Drupal 7)
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Dec 2006 at 17:48 UTC
Updated:
17 Mar 2007 at 22:16 UTC
I finally got core RSS generation patched to allow modules to change the item's body http://drupal.org/node/41703 but now I find out that views_rss.module had copied the old node code:
// Except for the original being a while and this being a foreach, this is
// completely cut & pasted from node.module.
Here's a patch to bring views into line with both 4.7 and HEAD's RSS code.
| Comment | File | Size | Author |
|---|---|---|---|
| views_rss.module.patch | 1.73 KB | drewish |
Comments
Comment #1
drewish commentedhold off on applying this to the 5.0 branch while i see if we can get this comitted: http://drupal.org/node/103371
it should be good to go on 4.7 though.
Comment #2
moshe weitzman commentedactually, i think we can use the new views_build_view() param that lets caller run its own query. then we pass the $result to node_feed() and get rid of this code duplication. untested, but works in my head.
Comment #3
merlinofchaos commentedCommitted to both branches to keep them in sync. drewish, if you get that patch in just open a new issue here.
Comment #4
merlinofchaos commentedWhoops, apparently I never set this fixed.
Comment #5
(not verified) commented