Change a poll type URL manually crashes
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Low
|
Unassigned |
Bug Description
Steps to reproduce (using sample data):
1.http://
2. Manually change the URL to: http://
3. Crash
* Module zope.app.
raise TraversalError(
_
TraversalError: (<zope.
Related branches
- Robert Collins (community): Approve
-
Diff: 322 lines (+79/-36)8 files modifiedlib/lp/registry/browser/configure.zcml (+3/-9)
lib/lp/registry/browser/poll.py (+15/-5)
lib/lp/registry/browser/tests/poll-views_0.txt (+1/-1)
lib/lp/registry/browser/tests/test_poll.py (+38/-0)
lib/lp/registry/stories/team-polls/edit-poll.txt (+1/-1)
lib/lp/registry/stories/team-polls/vote-poll.txt (+8/-8)
lib/lp/registry/stories/team-polls/xx-poll-condorcet-voting.txt (+10/-10)
lib/lp/testing/factory.py (+3/-2)
Changed in launchpad: | |
status: | Unconfirmed → Confirmed |
Changed in launchpad: | |
importance: | Medium → High |
affects: | launchpad-foundations → launchpad-registry |
description: | updated |
tags: | added: polls |
Changed in launchpad-registry: | |
milestone: | none → 10.11 |
assignee: | nobody → Curtis Hovey (sinzui) |
status: | Triaged → In Progress |
Changed in launchpad-registry: | |
status: | In Progress → Fix Committed |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad-registry: | |
status: | Fix Committed → Fix Released |
Changed in launchpad: | |
assignee: | Curtis Hovey (sinzui) → nobody |
We only have condorcet polls in our sampledata and it's not possible for users to create new condorcet polls, which means the +vote-condorcet page is not advertised anywhere.