From 9db327a69efb8734f22d5f4e22c50ed3702a40aa Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 29 Jan 2021 15:50:46 -0800
Subject: [PATCH] Edit view for site settings

---
 bookwyrm/templates/layout.html        | 10 ++++++++++
 bookwyrm/templates/settings/site.html |  4 ++--
 bookwyrm/views/site.py                | 11 ++++++++++-
 3 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/bookwyrm/templates/layout.html b/bookwyrm/templates/layout.html
index 3a33196c..81782970 100644
--- a/bookwyrm/templates/layout.html
+++ b/bookwyrm/templates/layout.html
@@ -90,6 +90,9 @@
                             Import books
                         </a>
                     </li>
+                    {% if perms.bookwyrm.create_invites or perms.bookwyrm.edit_instance_settings%}
+                    <hr class="navbar-divider">
+                    {% endif %}
                     {% if perms.bookwyrm.create_invites %}
                     <li>
                         <a href="{% url 'settings-invites' %}" class="navbar-item">
@@ -97,6 +100,13 @@
                         </a>
                     </li>
                     {% endif %}
+                    {% if perms.bookwyrm.edit_instance_settings %}
+                    <li>
+                        <a href="{% url 'settings-site' %}" class="navbar-item">
+                            Site Configuration
+                        </a>
+                    </li>
+                    {% endif %}
                     <hr class="navbar-divider">
                     <li>
                         <a href="/logout" class="navbar-item">
diff --git a/bookwyrm/templates/settings/site.html b/bookwyrm/templates/settings/site.html
index 830a7dc5..d071facf 100644
--- a/bookwyrm/templates/settings/site.html
+++ b/bookwyrm/templates/settings/site.html
@@ -51,11 +51,11 @@
         <h2 class="title is-4">Footer Content</h2>
         <div class="control">
             <label class="label" for="id_support_link">Support link:</label>
-            <input type="text" name="support_link" maxlength="255" class="input" id="id_support_link" placeholder="https://www.patreon.com/bookwyrm" value="{{ site.support_link }}">
+            <input type="text" name="support_link" maxlength="255" class="input" id="id_support_link" placeholder="https://www.patreon.com/bookwyrm"{% if site.support_link %} value="{{ site.support_link  }}"{% endif %}>
         </div>
         <div class="control">
             <label class="label" for="id_support_title">Support title:</label>
-            <input type="text" name="support_title" maxlength="100" class="input" id="id_support_title" placeholder="Patreon" value="{{ site.support_title }}">
+            <input type="text" name="support_title" maxlength="100" class="input" id="id_support_title" placeholder="Patreon"{% if site.support_title %} value="{{ site.support_title }}"{% endif %}>
         </div>
         <div class="control">
             <label class="label" for="id_admin_email">Admin email:</label>
diff --git a/bookwyrm/views/site.py b/bookwyrm/views/site.py
index 85c1751d..0a5e270b 100644
--- a/bookwyrm/views/site.py
+++ b/bookwyrm/views/site.py
@@ -27,5 +27,14 @@ class Site(View):
 
     def post(self, request):
         ''' edit the site settings '''
+        site = models.SiteSettings.objects.get()
+        form = forms.SiteForm(request.POST, instance=site)
+        if not form.is_valid():
+            data = {
+                'title': 'Site Settings',
+                'site_form': form
+            }
+            return TemplateResponse(request, 'settings/site.html', data)
+        form.save()
 
-        return redirect('/settings/site-settings')
+        return redirect('settings-site')