CGM_Public/pretix_cgo
1
0
Fork 0
forked from CGM_Public/pretix_original
Code Pull Requests Actions Activity

Fixed a problem with plugin URL reversal

Browse Source 
It turns out that django does not support multiple include() statements
that define the same namespace (only the first one will be used). But
there isn't a problem that can't be solved with another level of
indirection ;)
This commit is contained in:
Raphael Michel
2016-08-15 15:33:59 +02:00
parent 74866437f4
commit 0663bb902e
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/pretix/multidomain/maindomain_urlconf.py
View File

@@ -23,14 +23,15 @@ for app in apps.get_app_configs():
if hasattr(app, 'PretixPluginMeta'):
if importlib.util.find_spec(app.name + '.urls'):
urlmod = importlib.import_module(app.name + '.urls')
single_plugin_patterns = []
if hasattr(urlmod, 'urlpatterns'):
raw_plugin_patterns.append(
url(r'', include(urlmod, namespace=app.label))
)
single_plugin_patterns += urlmod.urlpatterns
if hasattr(urlmod, 'event_patterns'):
raw_plugin_patterns.append(
url(r'^(?P<organizer>[^/]+)/(?P<event>[^/]+)/', include(urlmod.event_patterns, namespace=app.label))
)
single_plugin_patterns.append(url(r'^(?P<organizer>[^/]+)/(?P<event>[^/]+)/',
include(urlmod.event_patterns)))
raw_plugin_patterns.append(
url(r'', include(single_plugin_patterns, namespace=app.label))
)
elif importlib.util.find_spec(app.name + '.maindomain_urls'):
warnings.warn('Please put your config in an \'urls\' module using the urlpatterns and event_patterns '
'attribute. Support for maindomain_urls in plugins will be dropped in the future.',