Linking a private project to an ubuntu source package breaks Ubuntu
Bug #1063297 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Aaron Bentley | ||
Bug Description
I can accidentally type the name of a private project when linking a package to a project, and I will get an Unauthorized error. If the project is shared with me, I can link the package to Ubuntu, breaking the ubuntu series, DSP and SP pages. From the private project's packaging page, i can link it to a source package.
Private projects cannot be linked to a package. The vocabularies cannot ever permit a private projects to be in this kind of relationship. The project packaging packages should not exist, or if they do, they need to be disabled.
Related branches
lp:~abentley/launchpad/no-proprietary-linked-products
- Deryck Hodge (community): Approve
-
Diff: 551 lines (+217/-16)16 files modifiedlib/lp/registry/browser/product.py (+7/-1)
lib/lp/registry/browser/productseries.py (+6/-2)
lib/lp/registry/browser/sourcepackage.py (+11/-1)
lib/lp/registry/browser/tests/test_product.py (+18/-1)
lib/lp/registry/browser/tests/test_productseries_views.py (+32/-1)
lib/lp/registry/browser/tests/test_sourcepackage_views.py (+38/-1)
lib/lp/registry/errors.py (+5/-0)
lib/lp/registry/model/packaging.py (+15/-3)
lib/lp/registry/model/product.py (+2/-0)
lib/lp/registry/model/productseries.py (+2/-2)
lib/lp/registry/model/sourcepackage.py (+2/-2)
lib/lp/registry/templates/productseries-index.pt (+2/-1)
lib/lp/registry/tests/test_packaging.py (+25/-0)
lib/lp/registry/tests/test_product.py (+12/-0)
lib/lp/registry/tests/test_productseries.py (+26/-1)
lib/lp/registry/tests/test_sourcepackage.py (+14/-0)
| Changed in launchpad: | |
| assignee: | nobody → Aaron Bentley (abentley) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r16142 <http://