+needspackaging bug counts are wrong
Bug #613610 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Unassigned | ||
Bug Description
+needspackaging should be showing the number of open bugs. It is showing the total of all bugs for the package. A quick examination shows that most of the packages list on the first page of results have no open bugs :(
Maybe we should consider not showing bugs and messag counts. We can show the package name and summary.
Related branches
lp:~sinzui/launchpad/dsp-bug-counts-0
- Brad Crittenden (community): Approve (code)
-
Diff: 153 lines (+26/-14)3 files modifiedlib/lp/bugs/tests/test_bugheat.py (+8/-1)
lib/lp/registry/model/distributionsourcepackage.py (+10/-9)
lib/lp/testing/factory.py (+8/-4)
lp:~sinzui/launchpad/dsp-bug-counts-1
- Deryck Hodge (community): Approve (code)
- Brad Crittenden (community): Approve (code)
-
Diff: 232 lines (+59/-14)5 files modifieddatabase/schema/security.cfg (+19/-11)
lib/lp/bugs/doc/bug-heat.txt (+0/-2)
lib/lp/bugs/model/bug.py (+3/-1)
lib/lp/bugs/tests/test_bugheat.py (+35/-0)
lib/lp/registry/model/distributionsourcepackage.py (+2/-0)
Revision history for this message
| Curtis Hovey (sinzui) wrote | #1 |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #2 |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #3 |
| Changed in launchpad-registry: | |
| assignee: | nobody → Curtis Hovey (sinzui) |
| status: | Triaged → In Progress |
| Changed in launchpad-registry: | |
| importance: | Low → High |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #4 |
| tags: | added: qa-needstesting |
| tags: |
added: qa-bad removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → In Progress |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-needstesting removed: qa-bad |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| assignee: | Curtis Hovey (sinzui) → nobody |
To post a comment you must log in.
DSP.recalculate
self.
I think this should only count open bugs. The orginal query that calculated total_bug_heat and bug_count used
bugtask.status in UNRESOLVED_
in the query to ensure closed bugs did not taint the priorities.
max_bug_heat may have always been counting all bugs, but I think this too should only be looking at open bugs.