This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/57/ |
On December 10th, 2013, 5:58 p.m. UTC, Tim Flink wrote:
blockerbugs/models/update.py (Diff revision 1) class Update(db.Model):55 order_by=date_submitted.desc()))55 order_by=(status.desc(), pending.desc())))Why is this sort order change an improvement? If an older update is pending while a newer update is stable, we'd want to show the newer updateOn December 11th, 2013, 2:59 p.m. UTC, Martin Krizek wrote:
But wouldn't the older update be marked as obsolete?On December 11th, 2013, 4:37 p.m. UTC, Tim Flink wrote:
It should be but for example, if you look at the tracking app right now, there are 2 anaconda/blivet updates - one of which has been deleted in bodhi but still shows up. This issue was supposed to be fixed but obviously, it's not. Filed as https://phab.qadevel.cloud.fedoraproject.org/T27 I think that the problem of phantom updates are the root cause of the problem as reported, but I'll ask in bug to be sure.On December 11th, 2013, 4:45 p.m. UTC, Tim Flink wrote:
After re-reading the bug, I stand corrected. That was a completely different issue than I was thinking about in my head. I don't like sorting on string fields by default but I also don't see much of a way around it unless we changed the schema to use status enums instead of strings. Go ahead and push to develop. We might want to look into changing the update status field to an enum but that's well outside the scope of this fix.
I know. This is just a part of the fix, the rest will be in https://phab.qadevel.cloud.fedoraproject.org/T21 as I mentioned in the fix description. Should have probably done it in one review, sorry for confusion.
- Martin
On December 10th, 2013, 3:54 p.m. UTC, Martin Krizek wrote:
Review request for blockerbugs.
By Martin Krizek.
Updated Dec. 10, 2013, 3:54 p.m.
Bugs:
438
Repository:
blockerbugs
Description
Testing
Diffs
|