Merge lp:~canonical-ca-hackers/ubuntu-webcatalog/1006442-double-distroseries into lp:ubuntu-webcatalog
Proposed by
Łukasz Czyżykowski
Status: | Merged |
---|---|
Approved by: | Michael Nelson |
Approved revision: | 135 |
Merged at revision: | 132 |
Proposed branch: | lp:~canonical-ca-hackers/ubuntu-webcatalog/1006442-double-distroseries |
Merge into: | lp:ubuntu-webcatalog |
Diff against target: |
67 lines (+31/-4) 3 files modified
src/webcatalog/models/applications.py (+3/-2) src/webcatalog/tests/test_views.py (+25/-0) src/webcatalog/views.py (+3/-2) |
To merge this branch: | bzr merge lp:~canonical-ca-hackers/ubuntu-webcatalog/1006442-double-distroseries |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Nelson (community) | Approve | ||
Review via email: mp+108937@code.launchpad.net |
Commit message
Cope with doubled applications in the database.
Description of the change
Overview
========
This branch fixes bug 1006442. It does that by coping with a rather strange condition in the database, because I wasn't able to find the root cause of why the current state was reached.
To post a comment you must log in.
14:11 < noodles> lukasz: could lines 31 and 41 do the client.get consistently, or are they doing different things?
14:12 < noodles> lukasz: actually, if they're doing the same thing, why do we need the first test? The one test (the second one) ensures that the status is 200 already?
14:12 < lukasz> noodles, yup, one is for app detail within distroseries, second is for general app detail
14:12 < noodles> lukasz: as, got it. +1