Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
node system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
8 Jul 2007 at 17:07 UTC
Updated:
30 Jul 2007 at 15:18 UTC
The node_feed() function allows modules to add namespaces to an RSS feed. However, they are added with array_merge in the loop while processing each node in the feed so the same namespace is added multiple times. The reason is that the namespaces array does not use keys so identical namespaces are added with unique integer keys. The attached bug fixes this. I highly recommend allowing this to backport to D5 as well.
I discovered this when I added the wfw namespace with changes to commentrss.module.
| Comment | File | Size | Author |
|---|---|---|---|
| node_feed_0.patch | 1.13 KB | tangent |
Comments
Comment #1
tangent commentedCan someone please comment on this? It's a very minimal patch.
Comment #2
dries commentedI've committed this to CVS HEAD. As this is a small API change, I suggest you document this in the upgrade instructions.
Thanks tangent.
Comment #3
tangent commentedI've noted the change as a comment here, as that's the only ability I have. Hopefully that's the documentation you are referring to.
http://drupal.org/node/114774#comment-252463
Comment #4
(not verified) commented