Re: Review Request 57: Multiple builds fixing a bug show confusing status label

[Martin Krizek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-tflink.rhcloud.com/r/57/
-----------------------------------------------------------

(Updated Dec. 16, 2013, 1:24 p.m.)


Status
------

This change has been marked as submitted.


Review request for blockerbugs.


Bugs: 438
    https://fedorahosted.org/fedora-qa/ticket/438


Repository: blockerbugs


Description
-------

commit 911781af9cc06e27c0a8580f07f7673c90e7329d
Author: Martin Krizek <mkrizek(a)redhat.com&gt;
Date:   Tue Dec 10 16:28:52 2013 +0100

    Sort updates by (status.desc(), pending.desc())
    
    This way an update label is correctly set when multiple updates fix a
    bug. The label is chosen by the following order (the lowest one):
    
    pending testing < testing < pending stable < stable
    
    Fixes: #438

Note: the rest of the issues with displaying label is handled outside this ticket in
https://phab.qadevel.cloud.fedoraproject.org/T21


Diffs
-----

  blockerbugs/models/update.py 89c083da5c91558d4dead118c67e4a1138b8cae1 
  blockerbugs/controllers/main.py 1795ed1d717bb85e0f3d9e7e7008d68c2aae36d8 
  blockerbugs/__init__.py dcf6c5aadb24e6d86e95d5bb3d608b81721de47f 

Diff: http://reviewboard-tflink.rhcloud.com/r/57/diff/


Testing
-------

Sorting seems to be working as expected.


Thanks,

Martin Krizek