Last Comment Bug 876790 - Use published date instead of updated
: Use published date instead of updated
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Feed Reader (show other bugs)
: 17
: All All
: -- normal (vote)
: Thunderbird 24.0
Assigned To: alta88
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-28 11:11 PDT by realname
Modified: 2013-06-25 05:22 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch (4.37 KB, patch)
2013-05-28 12:52 PDT, alta88
mkmelin+mozilla: review+
Details | Diff | Review

Description realname 2013-05-28 11:11:29 PDT
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:18.0) Gecko/20100101 Firefox/18.0 SeaMonkey/2.15.2 (Beta/Release)
Build ID: 20130203013604

Steps to reproduce:

Would be great if Thunderbird could use the published date in an Atom feed instead of the updated date if there's an empty one.

E. g. http://www.techstage.de/rss.xml

Contains for an entry the following elements:
		<published>2013-05-28T19:36:00+02:00</published>
		<updated></updated>

Unfortunately, Thunderbird is displaying the fetching time as article time. Would be great to have the published date as alternative as it sounds logical that a never-updated article has no updated-date.
Comment 1 alta88 2013-05-28 12:51:27 PDT
it's because there's an <updated> tag, but it's empty.  but yes, it would be nice to check that..
Comment 2 alta88 2013-05-28 12:52:38 PDT
Created attachment 754967 [details] [diff] [review]
patch


and `here it is.
Comment 3 Magnus Melin 2013-05-30 04:06:24 PDT
Comment on attachment 754967 [details] [diff] [review]
patch

Review of attachment 754967 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. r=mkmelin
Comment 4 Ryan VanderMeulen [:RyanVM] 2013-05-30 09:09:05 PDT
https://hg.mozilla.org/comm-central/rev/3a04eb8b4f83
Comment 5 realname 2013-05-30 09:40:41 PDT
Wow - many thanks for the fast implementation. Cannot wait to get the update ;)

Note You need to log in before you can comment on or make changes to this bug.