Merge lp:~pfalcon/lava-dashboard/1217880 into lp:lava-dashboard

Proposed by Paul Sokolovsky
Status: Merged
Merged at revision: 420
Proposed branch: lp:~pfalcon/lava-dashboard/1217880
Merge into: lp:lava-dashboard
Diff against target: 22 lines (+6/-6)
1 file modified
dashboard_app/urls.py (+6/-6)
To merge this branch: bzr merge lp:~pfalcon/lava-dashboard/1217880
Reviewer Review Type Date Requested Status
Antonio Terceiro Approve
Neil Williams Pending
Review via email: mp+184080@code.launchpad.net

Description of the change

Fixes lp:1217880. As hinted by mwhudson in the bug, it's just the matter of adjusting regexp. And I made it fully liberal - entire path component (anything between "/") is treated as a username.

To post a comment you must log in.
Revision history for this message
Antonio Terceiro (terceiro) wrote :

Looks good to me

 review approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'dashboard_app/urls.py'
--- dashboard_app/urls.py 2013-01-15 14:09:48 +0000
+++ dashboard_app/urls.py 2013-09-05 11:35:20 +0000
@@ -35,12 +35,12 @@
35 url(r'^filters/\+add-cases-for-test-json$', 'filters.views.filter_add_cases_for_test_json'),35 url(r'^filters/\+add-cases-for-test-json$', 'filters.views.filter_add_cases_for_test_json'),
36 url(r'^filters/\+attribute-name-completion-json$', 'filters.views.filter_attr_name_completion_json'),36 url(r'^filters/\+attribute-name-completion-json$', 'filters.views.filter_attr_name_completion_json'),
37 url(r'^filters/\+attribute-value-completion-json$', 'filters.views.filter_attr_value_completion_json'),37 url(r'^filters/\+attribute-value-completion-json$', 'filters.views.filter_attr_value_completion_json'),
38 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)$', 'filters.views.filter_detail'),38 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)$', 'filters.views.filter_detail'),
39 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)/json$', 'filters.views.filter_json'),39 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)/json$', 'filters.views.filter_json'),
40 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)/\+edit$', 'filters.views.filter_edit'),40 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)/\+edit$', 'filters.views.filter_edit'),
41 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)/\+subscribe$', 'filters.views.filter_subscribe'),41 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)/\+subscribe$', 'filters.views.filter_subscribe'),
42 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)/\+delete$', 'filters.views.filter_delete'),42 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)/\+delete$', 'filters.views.filter_delete'),
43 url(r'^filters/~(?P<username>[a-zA-Z0-9-_]+)/(?P<name>[a-zA-Z0-9-_]+)/\+compare/(?P<tag1>[a-zA-Z0-9-_: .]+)/(?P<tag2>[a-zA-Z0-9-_: .]+)$', 'filters.views.compare_matches'),43 url(r'^filters/~(?P<username>[^/]+)/(?P<name>[a-zA-Z0-9-_]+)/\+compare/(?P<tag1>[a-zA-Z0-9-_: .]+)/(?P<tag2>[a-zA-Z0-9-_: .]+)$', 'filters.views.compare_matches'),
44 url(r'^streams/$', 'bundle_stream_list'),44 url(r'^streams/$', 'bundle_stream_list'),
45 url(r'^streams/json$', 'bundle_stream_list_json'),45 url(r'^streams/json$', 'bundle_stream_list_json'),
46 url(r'^streams(?P<pathname>/[a-zA-Z0-9/._-]+)bundles/$', 'bundle_list'),46 url(r'^streams(?P<pathname>/[a-zA-Z0-9/._-]+)bundles/$', 'bundle_list'),

Subscribers

People subscribed via source and target branches