Merge lp://staging/~brian-murray/launchpad/bug-supervisor-nominate-for-release into lp://staging/launchpad

Proposed by Brian Murray
Status: Merged
Approved by: Aaron Bentley
Approved revision: no longer in the source branch.
Merged at revision: 11774
Proposed branch: lp://staging/~brian-murray/launchpad/bug-supervisor-nominate-for-release
Merge into: lp://staging/launchpad
Diff against target: 839 lines (+351/-124)
17 files modified
lib/canonical/launchpad/interfaces/ftests/validation.txt (+6/-2)
lib/canonical/launchpad/security.py (+24/-0)
lib/lp/bugs/browser/bug.py (+6/-3)
lib/lp/bugs/browser/tests/bug-nomination-views.txt (+22/-12)
lib/lp/bugs/browser/tests/bug-views.txt (+20/-6)
lib/lp/bugs/configure.zcml (+20/-20)
lib/lp/bugs/doc/bug-nomination.txt (+27/-14)
lib/lp/bugs/model/bug.py (+7/-0)
lib/lp/bugs/stories/bug-release-management/30-nominate-bug-for-distrorelease.txt (+45/-29)
lib/lp/bugs/stories/bug-release-management/40-nominate-bug-for-productseries.txt (+46/-33)
lib/lp/bugs/stories/bug-release-management/60-defer-product-bug.txt (+7/-1)
lib/lp/bugs/stories/bug-release-management/xx-anonymous-bug-nomination.txt (+1/-2)
lib/lp/bugs/stories/webservice/xx-bug.txt (+6/-0)
lib/lp/bugs/tests/bugs-emailinterface.txt (+23/-0)
lib/lp/bugs/tests/test_bugsupervisor_bugnomination.py (+85/-0)
lib/lp/registry/interfaces/distribution.py (+1/-1)
lib/lp/testing/factory.py (+5/-1)
To merge this branch: bzr merge lp://staging/~brian-murray/launchpad/bug-supervisor-nominate-for-release
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+38733@code.staging.launchpad.net

Commit message

Restrict the ability to nominate bugs for a release to bug supervisors, owners or drivers.

Description of the change

Bug 114766 is about making bug nominations more useful, for nomination reviewers, by restricting the number ability to nominate bugs for a release from anyone to the bug supervisor group. This is also something that the Ubuntu community is looking for as noted in bug 174375. I've setup the branch so the that only projects or distributions that have a bug supervisor set are affected by this change.

lib/lp/bugs/browser/bug.py:
   restrict the enabling of the +nominate link to the bug supervisor team if set
lib/lp/bugs/model/bug.py:
    added check to ensure that nominator is a member of the correct team
lib/lp/bugs/configure.zcml:
    sorted permission attributes alphabetically
lib/lp/testing/factorypy:
     add in a bug supervisor parameter to makeDistribution (one already exists for makeProject)
lib/lp/bugs/stories/bug-release-management/:
    modified tests 30- and 40- not to use sample data
    60- added in a nomination that used to be in 30- or 40-

 I added in test_bugsupervisor_bugnomination.py:
    bin/test -cvvt test_bugsupervisor_bugnomination
I've also modified the tests in bug-release-management:
    bin/test -cvvt bug-release-management

To post a comment you must log in.
Revision history for this message
Aaron Bentley (abentley) wrote :

Per IRC discussion, please change this so that the permission is only provided for bug supervisors.

review: Needs Fixing
Revision history for this message
Brian Murray (brian-murray) wrote :

I've updated the branch per our discussions so that only bug supervisors, drivers or owners see the +nominate link and can nominate a bug for a release. I also created 2 more permission classes NominateBugForProductSeries and NominateBugForDistroSeries. Also due to the changes in not allowing just anyone to nominate a bug for a release I had to modify a couple of tests (bug-nomination-views.txt and bug-nomination.txt).

Revision history for this message
Aaron Bentley (abentley) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.