Fixing checking list perms
This commit is contained in:
parent
99c1a670f4
commit
f13f3d460c
|
@ -329,11 +329,11 @@ def add_book(request):
|
||||||
@login_required
|
@login_required
|
||||||
def remove_book(request, list_id):
|
def remove_book(request, list_id):
|
||||||
"""remove a book from a list"""
|
"""remove a book from a list"""
|
||||||
with transaction.atomic():
|
book_list = get_object_or_404(models.List, id=list_id)
|
||||||
book_list = get_object_or_404(models.List, id=list_id)
|
item = get_object_or_404(models.ListItem, id=request.POST.get("item"))
|
||||||
item = get_object_or_404(models.ListItem, id=request.POST.get("item"))
|
item.raise_not_deletable(request.user)
|
||||||
item.raise_not_deletable(request.user)
|
|
||||||
|
|
||||||
|
with transaction.atomic():
|
||||||
deleted_order = item.order
|
deleted_order = item.order
|
||||||
item.delete()
|
item.delete()
|
||||||
normalize_book_list_ordering(book_list.id, start=deleted_order)
|
normalize_book_list_ordering(book_list.id, start=deleted_order)
|
||||||
|
@ -348,7 +348,7 @@ def set_book_position(request, list_item_id):
|
||||||
special care with the unique ordering per list.
|
special care with the unique ordering per list.
|
||||||
"""
|
"""
|
||||||
list_item = get_object_or_404(models.ListItem, id=list_item_id)
|
list_item = get_object_or_404(models.ListItem, id=list_item_id)
|
||||||
list_item.list.raise_not_editable(request.user)
|
list_item.book_list.raise_not_editable(request.user)
|
||||||
try:
|
try:
|
||||||
int_position = int(request.POST.get("position"))
|
int_position = int(request.POST.get("position"))
|
||||||
except ValueError:
|
except ValueError:
|
||||||
|
|
Loading…
Reference in New Issue