mirror of
https://github.com/bookwyrm-social/bookwyrm.git
synced 2025-03-30 07:05:29 +00:00
Catches places where form.save() needs the request
Thank you, tests.
This commit is contained in:
parent
8f79b362f8
commit
351292fcda
8 changed files with 10 additions and 10 deletions
|
@ -102,7 +102,7 @@ class EditAnnouncement(View):
|
|||
return TemplateResponse(
|
||||
request, "settings/announcements/edit_announcement.html", data
|
||||
)
|
||||
announcement = form.save()
|
||||
announcement = form.save(request)
|
||||
return redirect("settings-announcements", announcement.id)
|
||||
|
||||
|
||||
|
|
|
@ -39,7 +39,7 @@ class LinkDomain(View):
|
|||
"""Set display name"""
|
||||
domain = get_object_or_404(models.LinkDomain, id=domain_id)
|
||||
form = forms.LinkDomainForm(request.POST, instance=domain)
|
||||
form.save()
|
||||
form.save(request)
|
||||
return redirect("settings-link-domain", status=status)
|
||||
|
||||
|
||||
|
|
|
@ -68,7 +68,7 @@ class EditAuthor(View):
|
|||
if not form.is_valid():
|
||||
data = {"author": author, "form": form}
|
||||
return TemplateResponse(request, "author/edit_author.html", data)
|
||||
author = form.save()
|
||||
author = form.save(request)
|
||||
|
||||
return redirect(f"/author/{author.id}")
|
||||
|
||||
|
|
|
@ -30,7 +30,7 @@ class Feed(View):
|
|||
form = forms.FeedStatusTypesForm(request.POST, instance=request.user)
|
||||
if form.is_valid():
|
||||
# workaround to avoid broadcasting this change
|
||||
user = form.save(commit=False)
|
||||
user = form.save(request, commit=False)
|
||||
user.save(broadcast=False, update_fields=["feed_status_types"])
|
||||
filters_applied = True
|
||||
|
||||
|
|
|
@ -38,7 +38,7 @@ class GetStartedProfile(View):
|
|||
if not form.is_valid():
|
||||
data = {"form": form, "next": "get-started-books"}
|
||||
return TemplateResponse(request, "get_started/profile.html", data)
|
||||
save_user_form(form)
|
||||
save_user_form(request, form)
|
||||
return redirect(self.next_view)
|
||||
|
||||
|
||||
|
|
|
@ -195,7 +195,7 @@ def add_book(request):
|
|||
if not form.is_valid():
|
||||
return List().get(request, book_list.id, add_failed=True)
|
||||
|
||||
item = form.save(commit=False)
|
||||
item = form.save(request, commit=False)
|
||||
|
||||
if book_list.curation == "curated":
|
||||
# make a pending entry at the end of the list
|
||||
|
|
|
@ -34,14 +34,14 @@ class EditUser(View):
|
|||
data = {"form": form, "user": request.user}
|
||||
return TemplateResponse(request, "preferences/edit_user.html", data)
|
||||
|
||||
user = save_user_form(form)
|
||||
user = save_user_form(request, form)
|
||||
|
||||
return set_language(user, redirect("user-feed", request.user.localname))
|
||||
|
||||
|
||||
def save_user_form(form):
|
||||
def save_user_form(request, form):
|
||||
"""special handling for the user form"""
|
||||
user = form.save(commit=False)
|
||||
user = form.save(request, commit=False)
|
||||
|
||||
if "avatar" in form.files:
|
||||
# crop and resize avatar upload
|
||||
|
|
|
@ -33,7 +33,7 @@ class Report(View):
|
|||
if not form.is_valid():
|
||||
raise ValueError(form.errors)
|
||||
|
||||
report = form.save()
|
||||
report = form.save(request)
|
||||
if report.links.exists():
|
||||
# revert the domain to pending
|
||||
domain = report.links.first().domain
|
||||
|
|
Loading…
Reference in a new issue