From a2b1233afdc31b1af70ebf9e31d84810e9de7e04 Mon Sep 17 00:00:00 2001 From: Jason Ward Date: Fri, 17 Nov 2017 12:40:59 -0500 Subject: [PATCH 1/4] refs #1: Updated tox + travis. --- .travis.yml | 1 + tox.ini | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index ca07bf1..e616c83 100644 --- a/.travis.yml +++ b/.travis.yml @@ -10,6 +10,7 @@ env: - TOXENV=py33-dj18 - TOXENV=py34-dj18 - TOXENV=py35-dj18 + - TOXENV=py27-dj111 install: - pip install tox codecov script: tox -v diff --git a/tox.ini b/tox.ini index c8d52c7..9030fe8 100644 --- a/tox.ini +++ b/tox.ini @@ -3,7 +3,7 @@ skipsdist = True usedevelop = True minversion = 1.8 envlist = - py{27,33,34,35}-dj{18,19,110} + py{27,33,34,35}-dj{18,19,110,111} [testenv] basepython = @@ -17,3 +17,4 @@ deps = dj18: Django<1.9 dj19: Django<1.10 dj110: Django<1.11 + dj111: Django<1.12 From 9261bd5cef377073382e2f39c0fc188aee39c62b Mon Sep 17 00:00:00 2001 From: Jason Ward Date: Fri, 17 Nov 2017 12:41:31 -0500 Subject: [PATCH 2/4] refs #1; Fixed path to register. --- example/exampleapp/permissions.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/example/exampleapp/permissions.py b/example/exampleapp/permissions.py index 72263a1..14e9dcb 100644 --- a/example/exampleapp/permissions.py +++ b/example/exampleapp/permissions.py @@ -7,7 +7,7 @@ from authority.permissions import BasePermission class FlatPagePermission(BasePermission): """ This class contains a bunch of checks: - + 1. the default checks 'add_flatpage', 'browse_flatpage', 'change_flatpage' and 'delete_flatpage' 2. the custom checks: @@ -49,4 +49,4 @@ class FlatPagePermission(BasePermission): return False top_secret.short_description=_('Is allowed to see top secret flatpages') -authority.register(FlatPage, FlatPagePermission) +authority.sites.register(FlatPage, FlatPagePermission) From 173a12b48f0cc2ae006f1bc66900a71c36ac19c0 Mon Sep 17 00:00:00 2001 From: Jason Ward Date: Fri, 17 Nov 2017 12:41:58 -0500 Subject: [PATCH 3/4] refs #1: Updated urls.py --- example/urls.py | 47 +++++++++++++++++++++++++---------------------- 1 file changed, 25 insertions(+), 22 deletions(-) diff --git a/example/urls.py b/example/urls.py index 27a3498..5342183 100644 --- a/example/urls.py +++ b/example/urls.py @@ -1,39 +1,42 @@ -try: - from django.conf.urls import patterns, include, handler500, url -except ImportError: # django < 1.4 - from django.conf.urls.defaults import patterns, include, handler500, url +import django.contrib.auth.views +from django.conf.urls import include, handler500, url from django.conf import settings -from django.contrib import admin -import authority -admin.autodiscover() -authority.autodiscover() - -handler500 # Pyflakes +import authority.views +import authority.urls +import example.exampleapp.views from exampleapp.forms import SpecialUserPermissionForm -urlpatterns = patterns('', - (r'^admin/(.*)', admin.site.root), - #('^admin/', include(admin.site.urls)), - url(r'^authority/permission/add/(?P[\w\-]+)/(?P[\w\-]+)/(?P\d+)/$', - view='authority.views.add_permission', +authority.autodiscover() + +handler500 # flake8 + +urlpatterns = ( + url( + r'^authority/permission/add/(?P[\w\-]+)/(?P[\w\-]+)/(?P\d+)/$', # noqa + view=authority.views.add_permission, name="authority-add-permission", kwargs={'approved': True, 'form_class': SpecialUserPermissionForm} ), - url(r'^request/add/(?P[\w\-]+)/(?P[\w\-]+)/(?P\d+)/$', - view='authority.views.add_permission', + url( + r'^request/add/(?P[\w\-]+)/(?P[\w\-]+)/(?P\d+)/$', # noqa + view=authority.views.add_permission, name="authority-add-permission-request", kwargs={'approved': False, 'form_class': SpecialUserPermissionForm} ), - (r'^authority/', include('authority.urls')), - (r'^accounts/login/$', 'django.contrib.auth.views.login'), - url(r'^(?P[\/0-9A-Za-z]+)$', 'example.exampleapp.views.top_secret', {'lala': 'oh yeah!'}), + url(r'^authority/', include(authority.urls)), + url(r'^accounts/login/$', django.contrib.auth.views.login), + url( + r'^(?P[\/0-9A-Za-z]+)$', + example.exampleapp.views.top_secret, + {'lala': 'oh yeah!'}, + ), ) if settings.DEBUG: - urlpatterns += patterns('', - (r'^media/(?P.*)$', 'django.views.static.serve', { + urlpatterns += ( + url(r'^media/(?P.*)$', django.views.static.serve, { 'document_root': settings.MEDIA_ROOT, }), ) From 4f148b6c47abe3de41dc5077f2eb10e119c6ceea Mon Sep 17 00:00:00 2001 From: Jason Ward Date: Fri, 17 Nov 2017 12:43:01 -0500 Subject: [PATCH 4/4] refs #1: Added username field. Not really sure why it was needed, but whatever. --- example/users/models.py | 1 + 1 file changed, 1 insertion(+) diff --git a/example/users/models.py b/example/users/models.py index dc42bed..5ebd69b 100644 --- a/example/users/models.py +++ b/example/users/models.py @@ -7,6 +7,7 @@ class User(AbstractBaseUser, PermissionsMixin): USERNAME_FIELD = 'email' REQUIRED_FIELDS = ['first_name', 'last_name'] + username = models.CharField(max_length=100) first_name = models.CharField(max_length=50) last_name = models.CharField(max_length=50) email = models.EmailField(unique=True)