It seems Atom feeds from Ning use <updated> tags to hold the news item date... See http://bassingbourntransitionvillage.ning.com/notes/index/feed?type=all&... for an example.

Currently _parser_common_syndication_atom10_parse() in common_syndication_parser.inc assumes the date will be either 'published' or 'issued', but misses 'updated' - hence many feed items from Ning are always set to 'now'.

The attached patch looks for and uses 'updated' as a fall back.

(this works hand in hand with #1008384: Feeds not pulling publication date with feed using dc:date and RSS 2.0 (please commit!) to help ensure sensible date values are found)

CommentFileSizeAuthor
#3 atom-updated-1027020-3.patch890 bytesgetgood
Feeds_6.x-ATOM_parser_use_updated_date.patch1.42 KBjim kirkpatrick

Comments

Yura’s picture

Yura commented

Tested with 6.x-1.0-beta10 successfully. Managing News getting right pubdate from atom feeds now.

jim kirkpatrick’s picture

jim kirkpatrick commented
Status: Needs review » Reviewed & tested by the community
getgood’s picture

getgood commented
StatusFileSize
new atom-updated-1027020-3.patch890 bytes

This patch applies and fixes the same problem in 7.x-2.0-alpha4. I'm not sure what the convention is when it comes to creating an issue to forward port a patch. If anyone could lend some advice, I'd appreciate it.

Also, I can't find the "issued" tag anywhere in the ATOM specification. Is it there for legacy reasons, or has it just been forgotten? If it's not actually supported anymore, a better patch might be the one attached.

twistor’s picture

twistor commented
Status: Reviewed & tested by the community » Fixed

The original patch was applied to 7.x at some point long ago.
It was applied to 6.x here http://drupalcode.org/project/feeds.git/commit/d3f23b0.

Status: Fixed » Closed (fixed)

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