Activity feed pubDate tag showing oldest date instead of newest

[MrMaz]

Title says it all. I think this line should order by DESC not ASC

​http://trac.buddypress.org/browser/trunk/bp-activity/bp-activity-classes.php#L326

[johnjamesjacoby]

Can you show me a place on testbp.org where the activity shows up in the wrong order?

[MrMaz]

Sure. And actually, its not the sort order, its the pubDate tag in the feed header.

Load this up and view the XML source:
​http://testbp.org/activity/feed

Grep for "pubDate"

More info on pubDate in the spec:
​http://validator.w3.org/feed/docs/rss2.html

Technically your pubDate is supposed to be once or twice a day, but the bp activity class is using the date of the most recent update. I am not even sure if most feed readers use pubDate, but BP should try to follow the spec.

[MrMaz]

All BP activity feeds are breaking RSS spec because of this issue. Why not fix it asap?

I spent a lot of time researching this one, a comment would be appreciated.

[DJPaul]

MrMaz would you take a look at this please?

[r-a-y]

Re: ​http://trac.buddypress.org/ticket/1286#comment:5

Confirmed on testbp.org.
pubDate shows Jan 1, 1970.

[r-a-y]

Moving this to 1.2.6 because it's a valid issue and a simple fix.

AFAIK, the function, BP_Activity_Activity::get_last_updated() is only used for the activity RSS feeds.

[johnjamesjacoby]

(In [3115]) Fixes #1286 props MrMaz