Change a poll type URL manually crashes

Bug #46581 reported by Diogo Matsubara
6
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Unassigned

Bug Description

Steps to reproduce (using sample data):
1.http://launchpad.dev/~ubuntu-team/+poll/never-closes3/+vote-condorcet
2. Manually change the URL to: http://launchpad.dev/~ubuntu-team/+poll/never-closes3/+vote-simple
3. Crash

    * Module zope.app.traversing.adapters, line 61, in traverse
      raise TraversalError(subject, name)
      __traceback_info__: (<zope.app.pagetemplate.simpleviewclass.SimpleViewClass from /home/matsubara/devel/launchpad/oops-bugs/lib/canonical/launchpad/templates/poll-vote-simple.pt object at 0x43a9602c>, 'currentVote', ['option'])

TraversalError: (<zope.app.pagetemplate.simpleviewclass.SimpleViewClass from /home/matsubara/devel/launchpad/oops-bugs/lib/canonical/launchpad/templates/poll-vote-simple.pt object at 0x43a9602c>, 'currentVote')

Related branches

Changed in launchpad:
status: Unconfirmed → Confirmed
Christian Reis (kiko)
Changed in launchpad:
importance: Medium → High
Revision history for this message
Guilherme Salgado (salgado) wrote :

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.

Changed in launchpad:
importance: High → Low
Curtis Hovey (sinzui)
affects: launchpad-foundations → launchpad-registry
Curtis Hovey (sinzui)
description: updated
Revision history for this message
Curtis Hovey (sinzui) wrote :

Looks like the view needs to be split (condorcet subclasses the simple one) so that it can check that the poll type is the one expects.
and show an error if something goes wrong. The zcml is easy to split into multiple views.

Curtis Hovey (sinzui)
tags: added: polls
Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: none → 10.11
assignee: nobody → Curtis Hovey (sinzui)
status: Triaged → In Progress
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: In Progress → Fix Committed
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit
tags: added: qa-needstesting
Curtis Hovey (sinzui)
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui)
Changed in launchpad:
assignee: Curtis Hovey (sinzui) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.