OpenAustralia.org

RSS in wrong order?

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: Hackfest Sydney 2009
  • Component/s: Application
  • Labels:
    None

Description

Look at Robert McMullan's page:

http://www.openaustralia.org/mp/robert_mcmullan/fraser

Now look at the RSS for that page:

http://www.openaustralia.org/rss/mp/10450.rdf

It's different. Should be reverse chronologically ordered, same as the page.

Activity

Hide
Sherif Mansour added a comment - - edited

Confused.. looks the same order to me... can you elaborate?

Happy to work on this..

Show
Sherif Mansour added a comment - - edited Confused.. looks the same order to me... can you elaborate? Happy to work on this..
Hide
Andrew Bennetts added a comment -

Mary and I have found the bug, _get_data_by_person in hansardlist.php was sorting by date but not by time. I'll attach a patch.

Show
Andrew Bennetts added a comment - Mary and I have found the bug, _get_data_by_person in hansardlist.php was sorting by date but not by time. I'll attach a patch.
Hide
Andrew Bennetts added a comment -

This fixes this bug.

Show
Andrew Bennetts added a comment - This fixes this bug.
Hide
Andrew Bennetts added a comment -

Henare Degan has committed my patch to github, and will send it to Matthew (along with other changes) soon.

(See http://github.com/henare/twfy/commit/2c03d24d350ee98db3cd2a6b07ee8e8db2b363af)

Show
Andrew Bennetts added a comment - Henare Degan has committed my patch to github, and will send it to Matthew (along with other changes) soon. (See http://github.com/henare/twfy/commit/2c03d24d350ee98db3cd2a6b07ee8e8db2b363af)
Hide
Henare Degan added a comment -

Sending pull request to Matthew with Andrew's fix.

Show
Henare Degan added a comment - Sending pull request to Matthew with Andrew's fix.
Hide
Matthew Landauer added a comment -

Patch merged in at my end and deployed. Thanks for the fix!

Show
Matthew Landauer added a comment - Patch merged in at my end and deployed. Thanks for the fix!

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: