branchnamespace ignores sharing
Bug #1023424 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
William Grant | ||
Bug Description
The BranchNamespace class does not know about information-types, thus cannot know who the project has shared with.
Related branches
lp:~wgrant/launchpad/branchnamespace-information-type
- Curtis Hovey (community): Approve (code)
-
Diff: 696 lines (+137/-287)6 files modifiedlib/lp/code/errors.py (+0/-10)
lib/lp/code/interfaces/branchnamespace.py (+6/-16)
lib/lp/code/model/branch.py (+6/-9)
lib/lp/code/model/branchnamespace.py (+52/-76)
lib/lp/code/model/tests/test_branch.py (+5/-6)
lib/lp/code/model/tests/test_branchnamespace.py (+68/-170)
lp:~wgrant/launchpad/more-branchnamespace
- Ian Booth (community): Approve
-
Diff: 189 lines (+61/-40)5 files modifiedlib/lp/code/browser/branchlisting.py (+25/-6)
lib/lp/code/browser/tests/test_product.py (+13/-7)
lib/lp/code/interfaces/branchnamespace.py (+4/-6)
lib/lp/code/model/branchnamespace.py (+5/-8)
lib/lp/code/templates/product-branches.pt (+14/-13)
lp:~wgrant/launchpad/improve-branch-edit-type
- j.c.sackett (community): Approve
-
Diff: 531 lines (+185/-228)5 files modifiedlib/lp/code/browser/branch.py (+50/-25)
lib/lp/code/browser/tests/test_branch.py (+80/-58)
lib/lp/code/interfaces/branch.py (+5/-17)
lib/lp/code/model/branch.py (+14/-32)
lib/lp/code/model/tests/test_branch.py (+36/-96)
lp:~wgrant/launchpad/edit-stacked-information-type
- Ian Booth (community): Approve
-
Diff: 156 lines (+46/-59)2 files modifiedlib/lp/code/browser/branch.py (+28/-55)
lib/lp/code/browser/tests/test_branch.py (+18/-4)
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 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: |
added: qa-pl removed: qa-needstesting |
| tags: |
added: qa-ok removed: qa-pl |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r15613 <http://