From 3f280af715d800f8d8cb7ab610696abb226e8f41 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 10 Jan 2022 10:48:27 -0800
Subject: [PATCH] Functionality to edit name

---
 bookwyrm/forms.py                                     |  2 +-
 .../settings/link_domains/edit_domain_modal.html      |  4 ++--
 bookwyrm/urls.py                                      |  4 ++--
 bookwyrm/views/admin/link_domains.py                  | 11 ++++++++---
 4 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/bookwyrm/forms.py b/bookwyrm/forms.py
index 12ebbff8..5ad36501 100644
--- a/bookwyrm/forms.py
+++ b/bookwyrm/forms.py
@@ -219,7 +219,7 @@ class CoverForm(CustomForm):
 class LinkDomainForm(CustomForm):
     class Meta:
         model = models.LinkDomain
-        fields = ["id", "name"]
+        fields = ["name"]
 
 
 class FileLinkForm(CustomForm):
diff --git a/bookwyrm/templates/settings/link_domains/edit_domain_modal.html b/bookwyrm/templates/settings/link_domains/edit_domain_modal.html
index 7b2a46e2..984ad78d 100644
--- a/bookwyrm/templates/settings/link_domains/edit_domain_modal.html
+++ b/bookwyrm/templates/settings/link_domains/edit_domain_modal.html
@@ -6,14 +6,14 @@
 {% endblock %}
 
 {% block modal-form-open %}
-<form name="edit-domain-{{ domain.id }}" method="post" action="{% url 'settings-link-domain' %}">
+<form name="edit-domain-{{ domain.id }}" method="post" action="{% url 'settings-link-domain' status=status domain_id=domain.id %}">
 {% endblock %}
 
 {% block modal-body %}
 {% csrf_token %}
 <label class="label" for="id_name">{% trans "Name:" %}</label>
 <div class="control">
-    <input type="text" id="id_name" class="input" value="{{ domain.name }}">
+    <input type="text" id="id_name" class="input" name="name" value="{{ domain.name }}">
 </div>
 {% endblock %}
 
diff --git a/bookwyrm/urls.py b/bookwyrm/urls.py
index 781430dc..0d41e4f5 100644
--- a/bookwyrm/urls.py
+++ b/bookwyrm/urls.py
@@ -154,12 +154,12 @@ urlpatterns = [
         name="settings-email-blocks-delete",
     ),
     re_path(
-        r"^setting/link-domains/(?P<status>(pending|approved|blocked|))/?",
+        r"^setting/link-domains/(?P<status>(pending|approved|blocked))/?$",
         views.LinkDomain.as_view(),
         name="settings-link-domain",
     ),
     re_path(
-        r"^setting/link-domains/?",
+        r"^setting/link-domains/(?P<status>(pending|approved|blocked))/(?P<domain_id>\d+)/?$",
         views.LinkDomain.as_view(),
         name="settings-link-domain",
     ),
diff --git a/bookwyrm/views/admin/link_domains.py b/bookwyrm/views/admin/link_domains.py
index 0b662b85..4f11af11 100644
--- a/bookwyrm/views/admin/link_domains.py
+++ b/bookwyrm/views/admin/link_domains.py
@@ -1,5 +1,6 @@
 """ Manage link domains"""
 from django.contrib.auth.decorators import login_required, permission_required
+from django.shortcuts import get_object_or_404, redirect
 from django.template.response import TemplateResponse
 from django.utils.decorators import method_decorator
 from django.views import View
@@ -15,7 +16,7 @@ from bookwyrm import forms, models
 class LinkDomain(View):
     """Moderate links"""
 
-    def get(self, request, status="pending"):
+    def get(self, request, status):
         """view pending domains"""
         data = {
             "domains": models.LinkDomain.objects.filter(
@@ -28,5 +29,9 @@ class LinkDomain(View):
             request, "settings/link_domains/link_domains.html", data
         )
 
-    def post(self, request):
-        """post?"""
+    def post(self, request, status, domain_id):
+        """Set display name"""
+        domain = get_object_or_404(models.LinkDomain, id=domain_id)
+        form = forms.LinkDomainForm(request.POST, instance=domain)
+        form.save()
+        return redirect('settings-link-domain', status=status)