369672 - Feed preview doesn't show dates when the timezone is specified as "UTC"

Status: RESOLVED FIXED

(Firefox Graveyard :: RSS Discovery and Preview, defect)

Description

[Michael Ventnor]

Attachment: One-line fix

Click on the RSS feed in the URL. You'll notice that no dates are put into the preview, but when you view the source of the feed, dates are in fact given but with the timezone "UTC" rather than "UT" or "GMT". Our javascript Date() is fully capable of handling this so only our regular expression is stopping the date from being displayed.

I suspect other feeds use "UTC" as well.

Comment 1

[Robert Sayre]

Hi, I'm sorry I let this sit for so long. r=sayrer w/ a unit test added.

Comment 2

[Michael Ventnor]

Attachment: For checkin

Updated to tip

Comment 3

[Michael Ventnor]

Attachment: Unit test

CVS hates me so I'm just going to post the unit test here. This goes in toolkit/components/feeds/test/xml/rss2/

Comment 4

[Robert Sayre]

Checking in src/FeedProcessor.js;
/cvsroot/mozilla/toolkit/components/feeds/src/FeedProcessor.js,v  <--  FeedProcessor.js
new revision: 1.25; previous revision: 1.24
done
RCS file: /cvsroot/mozilla/toolkit/components/feeds/test/xml/rss2/feed_pubDate_utc.xml,v
done
Checking in test/xml/rss2/feed_pubDate_utc.xml;
/cvsroot/mozilla/toolkit/components/feeds/test/xml/rss2/feed_pubDate_utc.xml,v  <--  feed_pubDate_utc.xml
initial revision: 1.1
done