Plus sign in meeting name breaks url lookup

Bug #853991 reported by James Westby
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Summit
Fix Released
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.

Tags: linaro

Related branches

James Westby (james-w)
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)
Changed in summit:
status: Fix Committed → Fix Released
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.