From 0007547a7c1c3424b0d254a4707888b07f82deee Mon Sep 17 00:00:00 2001
From: Jason McBrayer <jmcbray@carcosa.net>
Date: Fri, 15 Feb 2019 17:42:36 -0500
Subject: [PATCH] Allow editing filters

---
 brutaldon/templates/filters/edit.html | 79 +++++++++++++++++++++++++++
 brutaldon/templates/filters/list.html |  7 ++-
 brutaldon/urls.py                     |  1 +
 brutaldon/views.py                    | 50 +++++++++++++++--
 4 files changed, 132 insertions(+), 5 deletions(-)
 create mode 100644 brutaldon/templates/filters/edit.html

diff --git a/brutaldon/templates/filters/edit.html b/brutaldon/templates/filters/edit.html
new file mode 100644
index 0000000..1af053b
--- /dev/null
+++ b/brutaldon/templates/filters/edit.html
@@ -0,0 +1,79 @@
+{% extends "base.html" %}
+{% load widget_tweaks %}
+
+{% block content %}
+
+    <div class="container">
+        <h1 class="title">Edit Filter</h1>
+
+        <form method="post" id="edit-filter-form"
+              action="{% url "edit_filter" filter.id %}">
+            {% csrf_token %}
+
+            <div>
+                {{ form.non_field_errors }}
+            </div>
+
+            <div class="field">
+                <label class="label"> {{ form.phrase.label }}</label>
+                <div class="control">
+                    {% render_field form.phrase class+="input" %}
+                </div>
+            </div>
+
+            <div class="columns">
+                <div class="column field">
+                    <label class="label checkbox">
+                        {{ form.context_home.label }}
+                        {% render_field form.context_home %}
+                    </label>
+                </div>
+                <div class="column field">
+                    <label class="label checkbox">
+                        {{ form.context_public.label }}
+                        {% render_field form.context_public %}
+                    </label>
+                </div>
+                <div class="column field">
+                    <label class="label checkbox">
+                        {{ form.context_notes.label }}
+                        {% render_field form.context_notes %}
+                    </label>
+                </div>
+                <div class="column field">
+                    <label class="label checkbox">
+                        {{ form.context_thread.label }}
+                        {% render_field form.context_thread %}
+                    </label>
+                </div>
+            </div>
+
+            <div class="field">
+                <label class="label checkbox">
+                    {{ form.whole_word.label }}
+                    {% render_field form.whole_word %}
+                </label>
+            </div>
+
+            <div class="field">
+                <label class="label" for="expires_in">{{ form.expires_in.label }}</label>
+                <div class="control has-icons-left">
+                    <div class="select">
+                        {% render_field form.expires_in class+="select" %}
+                        <span class="icon is-small is-left">
+                            <span class="fa fa-clock-o"></span>
+                        </span>
+                    </div>
+                </div>
+            </div>
+
+            <div class="field">
+                <input type="submit"  name="submit" value="Save"
+                       class="button is-primary">
+            </div>
+
+        </form>
+    </div>
+
+
+{% endblock %}
diff --git a/brutaldon/templates/filters/list.html b/brutaldon/templates/filters/list.html
index ef328ec..0539a17 100644
--- a/brutaldon/templates/filters/list.html
+++ b/brutaldon/templates/filters/list.html
@@ -23,7 +23,12 @@
                                 {{ context }}
                             {% endfor %}
                         </td>
-                        <td>Edit filter</td>
+                        <td>
+                            <a href="{% url "edit_filter" filter.id %}">
+                                <span class="fa fa-pencil"></span>
+                                Edit filter
+                            </a>
+                        </td>
                         <td>
                             <a href="{% url "delete_filter" filter.id %}"
                                ic-delete-from="{% url "delete_filter" filter.id %}",
diff --git a/brutaldon/urls.py b/brutaldon/urls.py
index 285bda2..37b4e97 100644
--- a/brutaldon/urls.py
+++ b/brutaldon/urls.py
@@ -63,5 +63,6 @@ urlpatterns = [
     path('filters/list', views.list_filters, name='list_filters'),
     path('filters/create', views.create_filter, name='create_filter'),
     path('filters/delete/<id>', views.delete_filter, name='delete_filter'),
+    path('filters/edit/<id>', views.edit_filter, name='edit_filter'),
     path('', views.home, name=''),
 ]
diff --git a/brutaldon/views.py b/brutaldon/views.py
index 01a2935..7fd8c02 100644
--- a/brutaldon/views.py
+++ b/brutaldon/views.py
@@ -975,10 +975,7 @@ def list_filters(request):
 
 @br_login_required
 def create_filter(request):
-    try:
-        account, mastodon = get_usercontext(request)
-    except NotLoggedInException:
-        return redirect(about)
+    account, mastodon = get_usercontext(request)
     if request.method == 'POST':
         form = FilterForm(request.POST)
         if form.is_valid():
@@ -1028,3 +1025,48 @@ def delete_filter(request, id):
                        "own_acct": request.session["user"],
                        "confirm_page": True,
                        "preferences": account.preferences})
+
+@br_login_required
+def edit_filter(request, id):
+    account, mastodon = get_usercontext(request)
+    filter = mastodon.filter(id)
+
+    contexts = []
+    if request.method == 'POST':
+        form = FilterForm(request.POST)
+        if form.is_valid():
+            if form.cleaned_data['context_home']:
+                contexts.append('home')
+            if form.cleaned_data['context_public']:
+                contexts.append('public')
+            if form.cleaned_data['context_notes']:
+                contexts.append('notifications')
+            if form.cleaned_data['context_thread']:
+                contexts.append('thread')
+            expires = form.cleaned_data['expires_in']
+            if expires == "":
+                expires = None
+            mastodon.filter_update(id, form.cleaned_data['phrase'],
+                                   contexts,
+                                   whole_word=form.cleaned_data['whole_word'],
+                                   expires_in=expires)
+            return redirect(list_filters)
+        else:
+            return render(request, 'filters/edit.html',
+                          { 'form': form,
+                            'account': account,
+                            'filter': filter,
+                            'preferences': account.preferences})
+    else:
+        contexts = []
+        form = FilterForm({'phrase': filter.phrase,
+                          'context_home': "home" in filter.context,
+                          'context_public': "public" in filter.context,
+                          'context_notes': "notifications" in filter.context,
+                          'context_thread': "thread" in filter.context,
+                          'whole_word': filter.whole_word})
+        return render(request, 'filters/edit.html',
+                      { 'form': form,
+                        'account': account,
+                        'filter': filter,
+                        'preferences': account.preferences})