On translate sharing details page, usage settings behaves badly with permission issues
Bug #758920 reported by
Ursula Junque
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Abel Deuring | ||
Bug Description
1) Go to https:/
2) Click on the edit icon to change "Translations are enabled on the upstream project." (or negative equivalent).
* matsubara & jml get a "Sorry, an error occurred while loading the form." (this is bad, you should look at it) (sorry.png)
* Ursinha gets the form, but when choosing 'Unknown', 'External' or 'Not Applicable' options in the form, an error box is displayed: "The following errors were encountered: 403 forbidden." (403.png)
Related branches
lp:~adeuring/launchpad/api-query-permissions-on-object
- Aaron Bentley (community): Approve
- Abel Deuring (community): Needs Resubmitting
- Robert Collins (community): Needs Fixing
-
Diff: 301 lines (+200/-2)6 files modifiedlib/lp/registry/interfaces/person.py (+27/-0)
lib/lp/registry/interfaces/sourcepackage.py (+15/-1)
lib/lp/registry/model/person.py (+12/-0)
lib/lp/registry/model/sourcepackage.py (+14/-0)
lib/lp/registry/tests/test_person.py (+46/-0)
lib/lp/registry/tests/test_sourcepackage.py (+86/-1)
lp:~abentley/launchpad/translation-sharing-details-permissions
- j.c.sackett (community): Approve
-
Diff: 529 lines (+254/-48)7 files modifiedlib/lp/registry/interfaces/sourcepackage.py (+11/-5)
lib/lp/registry/model/sourcepackage.py (+28/-7)
lib/lp/registry/tests/test_sourcepackage.py (+105/-25)
lib/lp/translations/browser/sourcepackage.py (+1/-1)
lib/lp/translations/browser/tests/test_sharing_details.py (+4/-0)
lib/lp/translations/javascript/sourcepackage_sharing_details.js (+27/-3)
lib/lp/translations/javascript/tests/test_sourcepackage_sharing_details.js (+78/-7)
Revision history for this message
| Ursula Junque (ursinha) wrote | #1 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| Changed in launchpad: | |
| assignee: | nobody → Abel Deuring (adeuring) |
| milestone: | none → 11.05 |
| status: | Triaged → In Progress |
| tags: | added: qa-untestable |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| tags: |
added: qa-needstesting removed: qa-untestable |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Данило Шеган (danilo) wrote | #5 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
r12889 in stable (http://