Plus sign in meeting name breaks url lookup

Bug #853991 reported by James Westby on 2011-09-19
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Summit
Critical
James Westby

Bug Description

Traceback (most recent call last):
  File "/srv/summit.ubuntu.com/summit/../summit/schedule/render.py", line 366, in debug
    return self.as_html()
  File "/srv/summit.ubuntu.com/summit/../summit/schedule/render.py", line 601, in as_html
    html += self.meeting_div(meeting)
  File "/srv/summit.ubuntu.com/summit/../summit/schedule/render.py", line 765, in meeting_div
    % (urlsafe(meeting.meeting_page_url), settings.MEDIA_URL
  File "/srv/summit.ubuntu.com/summit/../summit/schedule/models/meetingmodel.py", line 144, in get_meeting_page_url
    return reverse('summit.schedule.views.meeting', args=args)
  File "/usr/lib/pymodules/python2.6/django/core/urlresolvers.py", line 350, in reverse
    *args, **kwargs)))
  File "/usr/lib/pymodules/python2.6/django/core/urlresolvers.py", line 300, in reverse
    "arguments '%s' not found." % (lookup_view_s, args, kwargs))
NoReverseMatch: Reverse for 'summit.schedule.decorators.inner' with arguments '(u'uds-p', 19097, u'linaro-hackfest-val+infr-1')' and keyword arguments '{}' not found.

The "+" in the meeting name is what breaks this, as the urls.py doesn't include it
in the relevant regex.

Related branches

James Westby (james-w) on 2011-09-19
Changed in summit:
status: New → Triaged
importance: Undecided → Critical
assignee: nobody → James Westby (james-w)
Changed in summit:
status: Triaged → Fix Committed
James Westby (james-w) on 2011-09-19
Changed in summit:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers