Remove access when someone is unsubscribed from a bug or branch
Bug #997425 reported by
Ian Booth
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Ian Booth | ||
Bug Description
The new disclosure work decouples subscription from visibility, so you no longer need to be subscribed to a 'private' artifact to be able to see it. The old behaviour used to be that we you were unsubscribed from a 'private' artifact, you lost access to it also. We want to keep this behaviour for now to allow a managed transition into the new feature.
Related branches
lp:~wallyworld/launchpad/legacy-unsubscribe-revokes-access-997425
- Curtis Hovey (community): Approve (code)
-
Diff: 307 lines (+164/-11)6 files modifiedlib/lp/bugs/model/bug.py (+12/-0)
lib/lp/bugs/tests/test_bugvisibility.py (+46/-4)
lib/lp/registry/services/tests/test_sharingservice.py (+0/-7)
lib/lp/services/features/flags.py (+7/-0)
lib/lp/testing/fixture.py (+38/-0)
lib/lp/testing/tests/test_fixture.py (+61/-0)
lp:~wallyworld/launchpad/createAccessGrants-robust
- William Grant: Approve (code)
- Diff: 0 lines
lp:~stevenk/launchpad/branch-unsubscribe-revokes
- Curtis Hovey (community): Approve (code)
-
Diff: 49 lines (+21/-0)2 files modifiedlib/lp/code/model/branch.py (+7/-0)
lib/lp/code/model/tests/test_branchsubscription.py (+14/-0)
lp:~wgrant/launchpad/stop-using-legacy-bug-access
- Curtis Hovey (community): Approve (code)
-
Diff: 88 lines (+7/-31)4 files modifieddatabase/schema/security.cfg (+1/-1)
lib/lp/bugs/model/bug.py (+5/-7)
lib/lp/bugs/tests/test_bugvisibility.py (+1/-16)
lib/lp/services/features/flags.py (+0/-7)
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: bad-commit-15359 qa-bad removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
| tags: | added: sharing |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-bad |
| 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 r15359 <http://