From b7411fd6648651230cc4d3e6b7b06a73e37f99de Mon Sep 17 00:00:00 2001
From: Mark Veidemanis <m@zm.is>
Date: Wed, 27 Jul 2022 08:29:20 +0100
Subject: [PATCH] Implement form for editing network

---
 app/urls.py                                   | 11 +++
 core/templates/base.html                      |  1 +
 .../threshold/irc/network/channels.html       | 24 ++---
 .../threshold/irc/network/edit-network.html   | 34 +++++++
 .../manage/threshold/irc/network/info.html    | 98 ++++++++++--------
 .../manage/threshold/irc/network/relays.html  | 99 +++++++++----------
 core/views/dynamic/manage/threshold/irc.py    | 39 ++++++++
 7 files changed, 201 insertions(+), 105 deletions(-)
 create mode 100644 core/templates/dynamic/manage/threshold/irc/network/edit-network.html

diff --git a/app/urls.py b/app/urls.py
index 81c531e..706758b 100644
--- a/app/urls.py
+++ b/app/urls.py
@@ -40,6 +40,7 @@ from core.views.dynamic.insights import (
 from core.views.dynamic.manage.threshold.irc import (
     ThresholdIRCNetworkChannels,
     ThresholdIRCNetworkInfo,
+    ThresholdIRCNetworkInfoEdit,
     ThresholdIRCNetworkRelays,
     ThresholdIRCNetworks,
     ThresholdIRCStats,
@@ -103,6 +104,16 @@ urlpatterns = [
         ThresholdIRCNetworkInfo.as_view(),
         name="threshold_irc_network_info",
     ),
+    path(
+        "manage/threshold/irc/network/<str:net>/info/edit/",
+        ThresholdIRCNetworkInfoEdit.as_view(),
+        name="threshold_irc_network_info_edit",
+    ),
+    path(
+        "manage/threshold/irc/network/<str:net>/edit/",
+        ThresholdIRCNetworkInfoEdit.as_view(),
+        name="threshold_irc_network_edit",
+    ),
     path(
         "manage/threshold/irc/network/<str:net>/relays/",
         ThresholdIRCNetworkRelays.as_view(),
diff --git a/core/templates/base.html b/core/templates/base.html
index 30c7d2e..294b82e 100644
--- a/core/templates/base.html
+++ b/core/templates/base.html
@@ -12,6 +12,7 @@
     <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/@creativebulma/bulma-tooltip@1.2.0/dist/bulma-tooltip.min.css">
     <link rel="stylesheet" href="https://site-assets.fontawesome.com/releases/v6.1.1/css/all.css" />
     <script src="https://unpkg.com/htmx.org@1.8.0" integrity="sha384-cZuAZ+ZbwkNRnrKi05G/fjBX+azI9DNOkNYysZ0I/X5ZFgsmMiBXgDZof30F5ofc" crossorigin="anonymous"></script>
+    <script src="https://unpkg.com/htmx.org@1.8.0/dist/ext/remove-me.js"></script>
     <script>
       document.addEventListener('DOMContentLoaded', () => {
 
diff --git a/core/templates/dynamic/manage/threshold/irc/network/channels.html b/core/templates/dynamic/manage/threshold/irc/network/channels.html
index 5b92247..4fbabe8 100644
--- a/core/templates/dynamic/manage/threshold/irc/network/channels.html
+++ b/core/templates/dynamic/manage/threshold/irc/network/channels.html
@@ -1,10 +1,10 @@
 <div id="channels">
-    {% if channels is not None %}
-      <div class="content" style="max-height: 30em; overflow: auto;">
-        <div class="table-container">
-          <table class="table is-fullwidth is-hoverable">
-            <tbody>
-              {% for channel, info in channels.items %}
+  {% if channels is not None %}
+    <div class="content" style="max-height: 30em; overflow: auto;">
+      <div class="table-container">
+        <table class="table is-fullwidth is-hoverable">
+          <tbody>
+            {% for channel, info in channels.items %}
               <tr>
                 <td>
                   {{ channel }}
@@ -13,10 +13,10 @@
                   </span>
                 </td>
               </tr>
-              {% endfor %}
-            </tbody>
-          </table>
-        </div>
+            {% endfor %}
+          </tbody>
+        </table>
       </div>
-    {% endif %}
-  </div>
\ No newline at end of file
+    </div>
+  {% endif %}
+</div>
\ No newline at end of file
diff --git a/core/templates/dynamic/manage/threshold/irc/network/edit-network.html b/core/templates/dynamic/manage/threshold/irc/network/edit-network.html
new file mode 100644
index 0000000..d01e2f9
--- /dev/null
+++ b/core/templates/dynamic/manage/threshold/irc/network/edit-network.html
@@ -0,0 +1,34 @@
+<form
+hx-headers='{"X-CSRFToken": "{{ csrf_token }}"}'
+hx-post="{% url 'threshold_irc_network_edit' net %}"
+hx-target="this"
+hx-swap="outerHTML">
+  <div class="table-container">
+    <table class="table is-fullwidth is-hoverable">
+      <thead>
+        <th>attribute</th>
+        <th>value</th>
+      </thead>
+      <tbody>
+        {% for key, item in network.items %}
+          <tr>
+            <th>{{ key }}</th>
+            <td>
+              <div class="field">
+                <input type="text" name="{{ key }}" value="{{ item }}">
+              </div>
+            </td>
+          </tr>
+        {% endfor %}
+
+      </tbody>
+    </table>
+    <button class="button is-primary">Submit</button>
+    <button
+      class="button is-primary"
+      hx-headers='{"X-CSRFToken": "{{ csrf_token }}"}'
+      hx-post="{% url 'threshold_irc_network_info' net %}">
+      Cancel
+    </button>
+  </div>
+</form>
diff --git a/core/templates/dynamic/manage/threshold/irc/network/info.html b/core/templates/dynamic/manage/threshold/irc/network/info.html
index a6f4212..2737842 100644
--- a/core/templates/dynamic/manage/threshold/irc/network/info.html
+++ b/core/templates/dynamic/manage/threshold/irc/network/info.html
@@ -1,51 +1,65 @@
 <div id="info">
-    {% if network is not None %}
-      <div class="content" style="max-height: 30em; overflow: auto;">
-        <div class="table-container">
-          <table class="table is-fullwidth is-hoverable">
-            <thead>
-              <th>attribute</th>
-              <th>value</th>
-            </thead>
-            <tbody>
-              {% for key, item in network.items %}
-                <tr>
-                  <th>{{ key }}</th>
-                  <td>
-                    {% if key == 'security' %}
-                      {% if item == 'none' %}
-                        <span class="icon">
-                          <i class="fa-solid fa-lock-open" aria-hidden="true"></i>
-                        </span>
-                      {% elif item == 'ssl' %}
-                        <span class="icon">
-                          <i class="fa-solid fa-lock" aria-hidden="true"></i>
-                        </span>
-                      {% endif %}
-                    {% elif key == 'relays' %}
+  {% if message is not None %}
+    <div class="notification is-{{ class }}" hx-ext="remove-me" remove-me="3s">
+      {{ message }}
+    </div>
+  {% endif %}
+  {% if network is not None %}
+    <div class="content" style="max-height: 30em; overflow: auto;">
+      <div class="table-container">
+        <table class="table is-fullwidth is-hoverable">
+          <thead>
+            <th>attribute</th>
+            <th>value</th>
+          </thead>
+          <tbody>
+            {% for key, item in network.items %}
+              <tr>
+                <th>{{ key }}</th>
+                <td>
+                  {% if key == 'security' %}
+                    {% if item == 'none' %}
                       <span class="icon">
-                        <i class="fa-brands fa-unity"></i>
+                        <i class="fa-solid fa-lock-open" aria-hidden="true"></i>
                       </span>
-                    {% elif key == 'channels' %}
+                    {% elif item == 'ssl' %}
                       <span class="icon">
-                        <i class="fa-solid fa-hashtag"></i>
-                      </span>
-                    {% elif key == 'records' %}
-                      <span class="icon">
-                        <i class="fa-solid fa-album"></i>
-                      </span>
-                    {% elif key == 'host' %}
-                      <span class="icon">
-                        <i class="fa-solid fa-router"></i>
+                        <i class="fa-solid fa-lock" aria-hidden="true"></i>
                       </span>
                     {% endif %}
+                  {% elif key == 'relays' %}
+                    <span class="icon">
+                      <i class="fa-brands fa-unity"></i>
+                    </span>
+                  {% elif key == 'channels' %}
+                    <span class="icon">
+                      <i class="fa-solid fa-hashtag"></i>
+                    </span>
+                  {% elif key == 'records' %}
+                    <span class="icon">
+                      <i class="fa-solid fa-album"></i>
+                    </span>
+                  {% elif key == 'host' %}
+                    <span class="icon">
+                      <i class="fa-solid fa-router"></i>
+                    </span>
+                  {% endif %}
+                  {% if item is not None %}
                     {{ item }}
-                  </td>
-                </tr>
-              {% endfor %}
-            </tbody>
-          </table>
+                  {% endif %}
+                </td>
+              </tr>
+            {% endfor %}
+          </tbody>
+        </table>
+        <div hx-target="#info" hx-swap="outerHTML">
+          <button hx-get="{% url 'threshold_irc_network_info_edit' network.net %}" class="button is-primary is-fullwidth">
+            <span class="icon" data-tooltip="Edit">
+              <i class="fa-solid fa-pencil" aria-hidden="true"></i>
+            </span>
+          </button>
         </div>
       </div>
-    {% endif %}
-  </div>
\ No newline at end of file
+    </div>
+  {% endif %}
+</div>
\ No newline at end of file
diff --git a/core/templates/dynamic/manage/threshold/irc/network/relays.html b/core/templates/dynamic/manage/threshold/irc/network/relays.html
index 266a91a..80bb039 100644
--- a/core/templates/dynamic/manage/threshold/irc/network/relays.html
+++ b/core/templates/dynamic/manage/threshold/irc/network/relays.html
@@ -1,59 +1,56 @@
 <div id="relays">
-    {% if relays is not None %}
-      <div class="content" style="max-height: 30em; overflow: auto;">
-        <div class="table-container">
-          <table class="table is-fullwidth is-hoverable">
-            <thead>
-              <th>net</th>
-              <th>id</th>
-              <th>registered</th>
-              <th>enabled</th>
-              <th>
-                <span class="icon">
-                  <i class="fa-solid fa-hashtag"></i>
-                </span>
-              </th>
-              <th>nick</th>
-            </thead>
-            <tbody>
-              {% for relay in relays %}
-                <tr>
-                  <th>{{ relay.net }}</th>
-                  <td>{{ relay.id }}</td>
-                  <td>
-                    {% if relay.registered %}
-                      <span class="icon">
-                        <i class="fa-solid fa-check" aria-hidden="true"></i>
-                      </span>
-                    {% else %}
+  {% if relays is not None %}
+    <div class="content" style="max-height: 30em; overflow: auto;">
+      <div class="table-container">
+        <table class="table is-fullwidth is-hoverable">
+          <thead>
+            <th>id</th>
+            <th>registered</th>
+            <th>enabled</th>
+            <th>
+              <span class="icon">
+                <i class="fa-solid fa-hashtag"></i>
+              </span>
+            </th>
+            <th>nick</th>
+          </thead>
+          <tbody>
+            {% for relay in relays %}
+              <tr>
+                <td>{{ relay.id }}</td>
+                <td>
+                  {% if relay.registered %}
+                    <span class="icon">
+                      <i class="fa-solid fa-check" aria-hidden="true"></i>
+                    </span>
+                  {% else %}
                     <span class="icon">
                       <i class="fa-solid fa-xmark" aria-hidden="true"></i>
                     </span>
-                    {% endif %}
-                  </td>
-                  <td>
-                    {% if relay.enabled %}
-                      <span class="icon">
-                        <i class="fa-solid fa-check" aria-hidden="true"></i>
-                      </span>
-                    {% else %}
+                  {% endif %}
+                </td>
+                <td>
+                  {% if relay.enabled %}
+                    <span class="icon">
+                      <i class="fa-solid fa-check" aria-hidden="true"></i>
+                    </span>
+                  {% else %}
                     <span class="icon">
                       <i class="fa-solid fa-xmark" aria-hidden="true"></i>
                     </span>
-                    {% endif %}
-                  </td>
-                  <td>
-
-                    {{ relay.chans }}
-                  </td>
-                  <td>
-                    {{ relay.nick }}
-                  </td>
-                </tr>
-              {% endfor %}
-            </tbody>
-          </table>
-        </div>
+                  {% endif %}
+                </td>
+                <td>
+                  {{ relay.chans }}
+                </td>
+                <td>
+                  {{ relay.nick }}
+                </td>
+              </tr>
+            {% endfor %}
+          </tbody>
+        </table>
       </div>
-    {% endif %}
-  </div>
\ No newline at end of file
+    </div>
+  {% endif %}
+</div>
\ No newline at end of file
diff --git a/core/views/dynamic/manage/threshold/irc.py b/core/views/dynamic/manage/threshold/irc.py
index 9125f07..9906a46 100644
--- a/core/views/dynamic/manage/threshold/irc.py
+++ b/core/views/dynamic/manage/threshold/irc.py
@@ -1,5 +1,7 @@
 from django.shortcuts import render
 from django.views import View
+from rest_framework.parsers import FormParser
+from rest_framework.views import APIView
 
 from core.lib.manage.threshold import (
     get_irc_channels,
@@ -38,6 +40,34 @@ class ThresholdIRCNetworkInfo(SuperUserRequiredMixin, View):
         return render(request, self.template_name, context)
 
 
+class ThresholdIRCNetworkInfoEdit(SuperUserRequiredMixin, APIView):
+    template_name = "dynamic/manage/threshold/irc/network/edit-network.html"
+    parser_classes = [FormParser]
+
+    def get(self, request, net):
+        """
+        Return the form to edit a network.
+        """
+        network = get_irc_network(net)
+        editable = ["auth", "host", "last", "port", "security"]
+        context = {
+            "net": net,
+            "network": {k: v for k, v in network.items() if k in editable},
+        }
+        return render(request, self.template_name, context)
+
+    def post(self, request, net):
+        """
+        Actually edit the network.
+        Returns the info pane with a message about the success.
+        """
+        template_name = "dynamic/manage/threshold/irc/network/info.html"
+        network = get_irc_network(net)
+        context = {"network": network, "message": "Edited!", "class": "info"}
+        print("REQUEST DATA", net, request.data)
+        return render(request, template_name, context)
+
+
 class ThresholdIRCNetworkRelays(SuperUserRequiredMixin, View):
     template_name = "dynamic/manage/threshold/irc/network/relays.html"
 
@@ -54,3 +84,12 @@ class ThresholdIRCNetworkChannels(SuperUserRequiredMixin, View):
         channels = get_irc_channels(net)
         context = {"channels": channels["channels"]}
         return render(request, self.template_name, context)
+
+
+# CRUD stuff
+class ThresholdIRCNetworkEdit(SuperUserRequiredMixin, APIView):
+    """
+    Edit a network's attributes.
+    """
+
+    parser_classes = [FormParser]