Implement an IHasBugHeat interface
Bug #529846 reported by
Deryck Hodge
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Deryck Hodge |
Bug Description
I've done a hackish fix to have a method return None on max_bug_heat for IPerson to fix a failing test, but this needs a proper fix. This bug is a reminder that this should be done ASAP after 10.02.
Updated: We should create an IHasBugHeat interface per the comment suggestions on this bug.
Related branches
lp:~deryck/launchpad/has-bug-heat-interface-529846
Merged
into
lp:launchpad
- Guilherme Salgado (community): Approve
-
Diff: 530 lines (+124/-97)11 files modifiedlib/lp/bugs/interfaces/bugtarget.py (+15/-10)
lib/lp/bugs/model/bugtarget.py (+51/-46)
lib/lp/registry/configure.zcml (+15/-3)
lib/lp/registry/model/distribution.py (+7/-4)
lib/lp/registry/model/distributionsourcepackage.py (+6/-3)
lib/lp/registry/model/distroseries.py (+7/-5)
lib/lp/registry/model/person.py (+0/-10)
lib/lp/registry/model/product.py (+7/-4)
lib/lp/registry/model/productseries.py (+6/-5)
lib/lp/registry/model/project.py (+4/-3)
lib/lp/registry/model/sourcepackage.py (+6/-4)
Changed in malone: | |
status: | Triaged → In Progress |
assignee: | nobody → Deryck Hodge (deryck) |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in malone: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
On Mon, 01 Mar 2010 17:17:09 Deryck Hodge wrote:
> Public bug reported:
>
> I've done a hackish fix to have a method return None on max_bug_heat for
> IPerson, but this needs a proper fix with a DB patch. This bug is a
> reminder that this should be done ASAP after 10.02.
I strongly suggest that you don't do this unless there is a really good reason
for a person or team to have a max_bug_heat.
Instead change the interface of IHasBugs as mentioned in the code review.