Skip to content
This repository was archived by the owner on Apr 9, 2022. It is now read-only.

Commit 1d4166c

Browse files
Merge pull request #2 from jakemcdermott/handle-none-obj-permission
Handle none obj permission
2 parents 6a06ddf + 1b9b38b commit 1d4166c

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

server/server/api/permissions.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@ def has_permission(self, request, view):
1919
def has_object_permission(self, request, view, obj=None):
2020
"""Only the author can modify existing instances."""
2121
is_safe = request.method in SAFE_METHODS
22-
is_author = request.user == obj.author
22+
23+
try:
24+
is_author = request.user == obj.author
25+
except AttributeError:
26+
is_author = False
2327

2428
return is_safe or is_author or request.user.is_superuser

server/server/api/urls.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from django.conf.urls import url, include
21
from rest_framework.routers import DefaultRouter
32

43
from .views import (

0 commit comments

Comments
 (0)