Comment 2 for bug 1441729

Revision history for this message
Colin Watson (cjwatson) wrote :

12:08 <cjwatson> wgrant: I'm giving up on the StormRangeFactory bit for now. I do actually now have something that appears to work - haven't tried with launchpadlib, but the next_collection_link entries have elaborate-looking memos with full dates in them. The
                 tricky bit is that it breaks if you pass the "wrong" parameters, because e.g. BPN isn't in the resultset so SRF can't cope with a resultset ordered by BPN, and at the moment I'm inserting the ...
12:08 <cjwatson> ... range factory in the interface so it doesn't get to be parameter-sensitive.
12:08 <cjwatson> Maybe this is fixable with a DecoratedResultSet or something, but I've run past the end of my timebox for this.
12:10 <cjwatson> I'll just push up something today that implements order_by_date without that.
12:11 <wgrant> cjwatson: I think launchpadlib is a bit magical, since it supports slicing.
12:12 <wgrant> But yeah, SRF for the default sort order is awkward.
12:12 <wgrant> (and pointless, since it can't be sorted by an index)