urls: Migrate from patterns strings to plain lists of urls.

This commit is contained in:
Brett Smith 2017-11-03 11:47:21 -04:00
parent b79a3617f0
commit b2a006f49c
5 changed files with 63 additions and 114 deletions

View file

@ -1,8 +1,8 @@
from django.conf.urls import patterns, url, include from django.conf.urls import url, include
from conservancy.apps.blog.models import Entry, EntryTag # relative import from conservancy.apps.blog.models import Entry, EntryTag # relative import
from conservancy.apps.staff.models import Person from conservancy.apps.staff.models import Person
from datetime import datetime from datetime import datetime
from conservancy.apps.blog.views import last_name, BlogYearArchiveView, BlogMonthArchiveView, BlogDayArchiveView, BlogDateDetailView from conservancy.apps.blog.views import last_name, BlogYearArchiveView, BlogMonthArchiveView, BlogDayArchiveView, BlogDateDetailView, custom_index, query
extra_context = {} extra_context = {}
@ -12,23 +12,14 @@ info_dict = {
'extra_context': extra_context, 'extra_context': extra_context,
} }
# urlpatterns = patterns('django.views.generic.date_based', urlpatterns = [
urlpatterns = patterns('', url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', BlogDateDetailView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', 'object_detail', dict(info_dict, slug_field='slug')), url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', BlogDayArchiveView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', 'archive_day', info_dict), url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', BlogMonthArchiveView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', 'archive_month', info_dict), url(r'^(?P<year>\d{4})/$', BlogYearArchiveView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/$', 'archive_year', dict(info_dict, url(r'^/?$', custom_index, dict(info_dict, paginate_by=4)),
# make_object_list=True)), url(r'^query/$', query),
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', BlogDateDetailView.as_view(**info_dict)), ]
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', BlogDayArchiveView.as_view(**info_dict)),
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', BlogMonthArchiveView.as_view(**info_dict)),
(r'^(?P<year>\d{4})/$', BlogYearArchiveView.as_view(**info_dict)),
)
urlpatterns += patterns('conservancy.apps.blog.views',
(r'^/?$', 'custom_index', dict(info_dict, paginate_by=4)),
(r'^query/$', 'query'),
)
# Code to display authors and tags on each blog page # Code to display authors and tags on each blog page

View file

@ -1,6 +1,6 @@
from django.conf.urls import patterns, url, include from django.conf.urls import url, include
from conservancy.apps.contractpatch import views as cpatch_views
urlpatterns = patterns( urlpatterns = [
'', url(r'', cpatch_views.index),
(r'', 'conservancy.apps.contractpatch.views.index'), ]
)

View file

@ -17,10 +17,10 @@
# along with this program in a file in the toplevel directory called # along with this program in a file in the toplevel directory called
# "AGPLv3". If not, see <http://www.gnu.org/licenses/>. # "AGPLv3". If not, see <http://www.gnu.org/licenses/>.
from django.conf.urls import patterns, url, include from django.conf.urls import url, include
from django.conf import settings from django.conf import settings
from conservancy.apps.news.models import PressRelease, ExternalArticle from conservancy.apps.news.models import PressRelease, ExternalArticle
from conservancy.apps.news.views import NewsYearArchiveView, NewsMonthArchiveView, NewsDayArchiveView, NewsDateDetailView from conservancy.apps.news.views import NewsYearArchiveView, NewsMonthArchiveView, NewsDayArchiveView, NewsDateDetailView, listing
info_dict = { info_dict = {
'queryset': PressRelease.objects.all().filter(sites__id__exact=settings.SITE_ID), 'queryset': PressRelease.objects.all().filter(sites__id__exact=settings.SITE_ID),
@ -31,18 +31,10 @@ external_article_dict = {
'articles': ExternalArticle.objects.all() 'articles': ExternalArticle.objects.all()
} }
urlpatterns = patterns('', urlpatterns = [
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', 'conservancy.apps.news.views.object_detail', info_dict), url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', NewsDateDetailView.as_view(**info_dict)),
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/(?P<slug>[-\w]+)/$', NewsDateDetailView.as_view(**info_dict)), url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', NewsDayArchiveView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', 'conservancy.apps.news.views.archive_day', info_dict), url(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', NewsMonthArchiveView.as_view(**info_dict)),
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/(?P<day>\w{1,2})/$', NewsDayArchiveView.as_view(**info_dict)), url(r'^(?P<year>\d{4})/$', NewsYearArchiveView.as_view(**info_dict)),
# (r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', 'conservancy.apps.news.views.archive_month', info_dict), url(r'^/?$', listing, dict(info_dict, paginate_by=6)),
(r'^(?P<year>\d{4})/(?P<month>[a-z]{3})/$', NewsMonthArchiveView.as_view(**info_dict)), ]
# (r'^(?P<year>\d{4})/$', 'conservancy.apps.news.views.archive_year',
# dict(info_dict, make_object_list=True)),
(r'^(?P<year>\d{4})/$', NewsYearArchiveView.as_view(**info_dict)),
)
urlpatterns += patterns('',
(r'^/?$', 'conservancy.apps.news.views.listing', dict(info_dict, paginate_by=6)),
)

View file

@ -1,11 +1,11 @@
from django.conf.urls import patterns from django.conf.urls import url
from conservancy.apps.supporter import views as supp_views
from conservancy.static import views as static_views
INDEX_VIEW = 'conservancy.apps.supporter.views.index' INDEX_VIEW = supp_views.index
pattern_pairs = [(r'^/?$', INDEX_VIEW)] urlpatterns = [url(r'^/?$', INDEX_VIEW)]
pattern_pairs.extend( urlpatterns.extend(
(r'^{}(?:\.html|/|)$'.format(basename), INDEX_VIEW) url(r'^{}(?:\.html|/|)$'.format(basename), INDEX_VIEW)
for basename in ['index', '2015-supporter-appeal', '2016-supporter-appeal'] for basename in ['index', '2015-supporter-appeal', '2016-supporter-appeal']
) )
pattern_pairs.append((r'', 'conservancy.static.views.index')) urlpatterns.append(url(r'', static_views.index))
urlpatterns = patterns('', *pattern_pairs)

View file

@ -17,75 +17,41 @@
# along with this program in a file in the toplevel directory called # along with this program in a file in the toplevel directory called
# "AGPLv3". If not, see <http://www.gnu.org/licenses/>. # "AGPLv3". If not, see <http://www.gnu.org/licenses/>.
from django.conf.urls import patterns, url, include from django.conf.urls import url, include
from django.contrib import admin from django.contrib import admin, admindocs
# import conservancy.settings from conservancy import feeds, frontpage, sponsors
from django.conf import settings import conservancy.apps.fundgoal.views as fundgoal_views
from conservancy.feeds import BlogFeed, PressReleaseFeed, OmnibusFeed import conservancy.static.views as static_views
# from django.views.static import serve
# from django.conf.urls.static import static
# from django.contrib.staticfiles.urls import staticfiles_urlpatterns
# import conservancy.static.overview.views
# handler404 = 'modpythoncustom.view404'
# handler401 = 'conservancy.static.views.handler401'
# handler403 = 'conservancy.static.views.handler403'
handler404 = 'conservancy.static.views.handler404'
# handler500 = 'conservancy.static.views.handler500'
admin.autodiscover() admin.autodiscover()
urlpatterns = patterns('', urlpatterns = [
(r'^$', 'conservancy.frontpage.view'), url(r'^$', frontpage.view),
(r'^sponsors$', 'conservancy.frontpage.view'), url(r'^sponsors$', frontpage.view),
(r'^sponsors/$', 'conservancy.sponsors.view'), url(r'^sponsors/$', sponsors.view),
(r'^sponsors/index.html$', 'conservancy.sponsors.view'), url(r'^sponsors/index.html$', sponsors.view),
(r'^admin/doc/', include('django.contrib.admindocs.urls')), url(r'^admin/doc/', include('django.contrib.admindocs.urls')),
(r'^admin/', admin.site.urls), url(r'^admin/', admin.site.urls),
(r'^feeds/blog/?$', BlogFeed()), url(r'^feeds/blog/?$', feeds.BlogFeed()),
(r'^feeds/news/?$', PressReleaseFeed()), url(r'^feeds/news/?$', feeds.PressReleaseFeed()),
(r'^feeds/omnibus/?$', OmnibusFeed()), url(r'^feeds/omnibus/?$', feeds.OmnibusFeed()),
(r'^feeds/?$', 'conservancy.feeds.view'), url(r'^feeds/?$', feeds.view),
(r'^news(/|$)', include('conservancy.apps.news.urls')), url(r'^news(/|$)', include('conservancy.apps.news.urls')),
(r'^blog(/|$)', include('conservancy.apps.blog.urls')), url(r'^blog(/|$)', include('conservancy.apps.blog.urls')),
# formerly static templated things... (dirs with templates) # formerly static templated things... (dirs with templates)
(r'^error/(40[134]|500)(?:/index\.html|/|)$', 'conservancy.static.views.handler'), url(r'^error/(40[134]|500)(?:/index\.html|/|)$', static_views.handler),
(r'^error', 'conservancy.static.views.index'), url(r'^error', static_views.index),
(r'^about', 'conservancy.static.views.index'), url(r'^about', static_views.index),
(r'^donate', 'conservancy.static.views.index'), url(r'^donate', static_views.index),
(r'^copyleft-compliance', 'conservancy.static.views.index', url(r'^copyleft-compliance', static_views.index,
{'fundraiser_sought' : 'vmware-match-0'}), {'fundraiser_sought' : 'vmware-match-0'}),
(r'^projects', 'conservancy.static.views.index'), url(r'^projects', static_views.index),
(r'^npoacct', 'conservancy.static.views.index', url(r'^npoacct', static_views.index,
{'fundraiser_sought' : 'npoacct'}), {'fundraiser_sought' : 'npoacct'}),
(r'^contractpatch', include('conservancy.apps.contractpatch.urls')), url(r'^contractpatch', include('conservancy.apps.contractpatch.urls')),
(r'^overview', 'conservancy.static.views.index'), url(r'^overview', static_views.index),
(r'^privacy-policy', 'conservancy.static.views.index'), url(r'^privacy-policy', static_views.index),
(r'^supporter', include('conservancy.apps.supporter.urls')), url(r'^supporter', include('conservancy.apps.supporter.urls')),
(r'^fundraiser_data', 'conservancy.apps.fundgoal.views.view'), url(r'^fundraiser_data', fundgoal_views.view),
) ]
# urlpatterns += url(regex = r'^%s(?P<path>.*)$' % conservancy.settings.STATIC_URL[1:],
# urlpatterns += url(regex = r'^/overview',
# view = 'django.views.static.serve',
# kwargs = {'document_root': conservancy.settings.STATIC_ROOT,
# 'show_indexes' : True})
# urlpatterns += (r'^(?P<path>.*)$', 'django.views.static.serve',
# urlpatterns += (r'^overview/$', 'django.views.static.serve',
# {'document_root': conservancy.settings.STATIC_ROOT,
# 'show_indexes' : True})
# https://docs.djangoproject.com/en/1.7/howto/static-files/
# + static(conservancy.settings.STATIC_URL, document_root=conservancy.settings.STATIC_ROOT)
# urlpatterns += staticfiles_urlpatterns()
# urlpatterns += static(settings.STATIC_URL, view='django.contrib.staticfiles.views.serve',
# urlpatterns += static('/', view='django.contrib.staticfiles.views.serve',
# document_root=settings.STATIC_ROOT,
# show_indexes=True)