Small fix on list item delete

bjdixon · Apr 10, 2014 · 3236ea4 · 3236ea4
1 parent 00df931
commit 3236ea4
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -7,9 +7,5 @@ Create lists of stuff with django.
 
 Built using Harry Percival's book Test Driven Web Development with Python. 
 
-Todo:
-- Only people logged in can create lists. 
-- Only list owners can view their lists.
-- At the moment anyone that's logged in can delete list items from any list
 
 
diff --git a/functional_tests/__pycache__/test_simple_list_creation.cpython-33.pyc b/functional_tests/__pycache__/test_simple_list_creation.cpython-33.pyc
diff --git a/lists/views.py b/lists/views.py
@@ -43,10 +43,10 @@ def add_item(request, list_id):
 def delete_item(request, item_id):
 	list_item = Item.objects.get(id=item_id)
 	list_ = list_item.list
-	if isinstance(request.user, AnonymousUser) and request.user is not list_.owner:
-			return redirect('/404_page_does_not_exist/')
-	list_item.delete()
-	return redirect('/lists/%d/' % (list_.id,))
+	if request.user == list_.owner:
+		list_item.delete()
+		return redirect('/lists/%d/' % (list_.id,))
+	return redirect('/404_page_does_not_exist/')
 
 def my_lists(request, email):
 	owner = User.objects.get(email=email)