BugSummary assumes that subscription == visibility
Bug #1017748 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
William Grant | ||
Bug Description
BugSummary still assumes that bug privacy is handled only through subscription. Its viewed_by column needs to be replaced with something that handles access policies and access grants, and it needs to be rebuilt.
Related branches
lp:~wgrant/launchpad/bugsummary-v2-rebuild
- Curtis Hovey (community): Approve (code)
-
Diff: 770 lines (+748/-0)4 files modifieddatabase/schema/security.cfg (+15/-0)
lib/lp/bugs/scripts/bugsummaryrebuild.py (+356/-0)
lib/lp/bugs/scripts/tests/test_bugsummaryrebuild.py (+347/-0)
scripts/bugsummary-rebuild.py (+30/-0)
lp:~wgrant/launchpad/bugsummary-v2-db-0
- Stuart Bishop (community): Approve (db)
- Robert Collins: Pending (db) requested
-
Diff: 668 lines (+194/-298)2 files modifieddatabase/schema/patch-2209-19-0.sql (+186/-290)
database/schema/security.cfg (+8/-8)
lp:~wgrant/launchpad/bugsummary-v2-apg-db
- Stuart Bishop (community): Approve (db)
- Robert Collins: Pending (db) requested
-
Diff: 759 lines (+282/-232)5 files modifieddatabase/sampledata/current-dev.sql (+101/-101)
database/sampledata/current.sql (+101/-101)
database/schema/patch-2209-19-3.sql (+49/-26)
lib/lp/bugs/doc/bugsummary.txt (+3/-1)
lib/lp/bugs/model/tests/test_bugsummary.py (+28/-3)
lp:~wgrant/launchpad/bugsummary-v2-apg-app
- Curtis Hovey (community): Approve (code)
-
Diff: 572 lines (+202/-138)5 files modifiedlib/lp/bugs/doc/bugsummary.txt (+107/-78)
lib/lp/bugs/model/bug.py (+13/-21)
lib/lp/bugs/model/bugsummary.py (+61/-3)
lib/lp/bugs/model/bugtask.py (+11/-21)
lib/lp/bugs/model/tests/test_bugsummary.py (+10/-15)
lp:~wgrant/launchpad/bugsummary-v2-rebuild-ap
- Benji York (community): Approve (code)
-
Diff: 273 lines (+76/-45)2 files modifiedlib/lp/bugs/scripts/bugsummaryrebuild.py (+31/-14)
lib/lp/bugs/scripts/tests/test_bugsummaryrebuild.py (+45/-31)
| Changed in launchpad: | |
| 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: hardening |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| William Grant (wgrant) wrote | #5 |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r15490 <http://