Consolidate forms for task creation and editing

todo/forms.py

@@ -19,16 +19,17 @@ class AddTaskListForm(ModelForm):
         exclude = []
 
 
-class AddItemForm(ModelForm):
+class AddEditItemForm(ModelForm):
     """The picklist showing the users to which a new task can be assigned
     must find other members of the groups the current list belongs to."""
 
-    def __init__(self, task_list, *args, **kwargs):
+    def __init__(self, user, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields['assigned_to'].queryset = get_user_model().objects.filter(groups__in=[task_list.group])
+        self.fields['assigned_to'].queryset = get_user_model().objects.filter(groups__in=user.groups.all())
         self.fields['assigned_to'].label_from_instance = lambda obj: "%s (%s)" % (obj.get_full_name(), obj.username)
         self.fields['assigned_to'].widget.attrs = {
             'id': 'id_assigned_to', 'class': "custom-select mb-3", 'name': 'assigned_to'}
+        self.fields['task_list'].value = kwargs['initial']['task_list'].id
 
     due_date = forms.DateField(
             widget=forms.DateInput(attrs={'type': 'date'}), required=False)
@@ -44,22 +45,6 @@ class AddItemForm(ModelForm):
         exclude = []
 
 
-class EditItemForm(ModelForm):
-    """The picklist showing the users to which a new task can be assigned
-    must find other members of the groups the current list belongs to."""
-
-    def __init__(self, *args, **kwargs):
-        super(EditItemForm, self).__init__(*args, **kwargs)
-        self.fields['assigned_to'].queryset = get_user_model().objects.filter(
-            groups__in=[self.instance.task_list.group])
-
-    due_date = forms.DateField(widget=forms.DateInput(attrs={'type': 'date'}), required=False)
-
-    class Meta:
-        model = Item
-        exclude = ('created_date', 'created_by',)
-
-
 class AddExternalItemForm(ModelForm):
     """Form to allow users who are not part of the GTD system to file a ticket."""
 

todo/templates/todo/include/task_edit.html

@@ -0,0 +1,54 @@
+{# Form used by both Add Task and Edit Task views #}
+
+<form action="" name="add_task" method="post">
+  {% csrf_token %}
+
+  <div id="AddEditTask" class="collapse mt-3">
+    <div class="form-group">
+      <label for="id_title" name="title">Task</label>
+      <input type="text" class="form-control" id="id_title" name="title" required placeholder="Task title"
+        value="{% if form.title.value %}{{ form.title.value }}{% endif %}">
+    </div>
+
+    <div class="form-group">
+      <label for="id_note">Description</label>
+      <textarea class="form-control" id="id_note" name="note" rows="5"
+        aria-describedby="inputNoteHelp">{% if form.note.value %}{{ form.note.value }}{% endif %}</textarea>
+      <small id="inputNoteHelp" class="form-text text-muted">
+        Describe the task or bug. Provide steps to reproduce the issue.
+      </small>
+    </div>
+
+    <div class="form-group">
+      <label for="id_due_date">Due Date</label>
+      <input type="date" class="form-control" id="id_due_date" name="due_date"
+        value="{% if form.due_date.value %}{{ form.due_date.value|date:"Y-m-d" }}{% endif %}">
+    </div>
+
+    <div class="form-group">
+      <label for="id_assigned_to">Assigned To</label>
+      {# See todo.forms.AddItemForm #}
+      {{form.assigned_to}}
+    </div>
+
+    <div class="form-group">
+      <label for="id_notify">Notify</label>
+      <input type="checkbox" checked="checked" class="form-control" id="id_notify" name="notify" aria-describedby="inputNotifyHelp"
+        value="{{ form.notify.text }}">
+      <small id="inputNotifyHelp" class="form-text text-muted">
+        Email notifications will only be sent if task is assigned to someone other than yourself.
+      </small>
+    </div>
+
+    <input type="hidden" name="priority"
+      value="{% if form.priority.value %}{{ form.priority.value }}{% else %}999{% endif %}" id="id_priority">
+    <input type="hidden" name="created_by" value="{{ request.user.id }}" id="id_created_by">
+    <input type="hidden" name="task_list" value="{{ form.task_list.value }}" id="id_task_list">
+    <input type="hidden" name="created_date" value="{{ created_date }}" id="id_created_date">
+
+    <p>
+      <input type="submit" name="add_edit_task" value="Submit" class="btn btn-primary">
+    </p>
+
+  </div>
+</form>

todo/templates/todo/list_detail.html

@@ -6,49 +6,11 @@
 {% block content %}
 
   {% if list_slug != "mine" %}
-    <form action="" name="add_task" method="post">
-      {% csrf_token %}
-
     <button class="btn btn-primary" id="AddTaskButton" type="button"
-        data-toggle="collapse" data-target="#AddTask">Add Task</button>
+      data-toggle="collapse" data-target="#AddEditTask">Add Task</button>
 
-      <div id="AddTask" class="collapse mt-3">
+    {# Task edit / new task form #}
-          <div class="form-group">
+    {% include 'todo/include/task_edit.html' %}
-            <label for="id_title" name="title">Task</label>
-            <input type="text" class="form-control" id="id_title" name="title" required placeholder="Task title" value="{{ form.title.text }}">
-          </div>
-
-          <div class="form-group">
-            <label for="id_note">Description</label>
-            <textarea class="form-control" id="id_note" name="note"
-              rows="5" aria-describedby="inputNoteHelp">{{ form.note.text }}</textarea>
-            <small id="inputNoteHelp" class="form-text text-muted">Describe the task or bug. Provide steps to reproduce the issue.</small>
-          </div>
-
-          <div class="form-group">
-              <label for="id_due_date">Due Date</label>
-              <input type="date" class="form-control" id="id_due_date" name="due_date" value="{{ form.due_date.text }}">
-          </div>
-
-          <div class="form-group">
-              <label for="id_assigned_to">Assigned To</label>
-              {# See todo.forms.AddItemForm #}
-              {{form.assigned_to}}
-          </div>
-
-          <div class="form-group">
-              <label for="id_notify">Notify</label>
-              <input type="checkbox" checked="checked" class="form-control" id="id_notify" name="notify" aria-describedby="inputNotifyHelp" value="{{ form.notify.text }}">
-              <small id="inputNotifyHelp" class="form-text text-muted">Email notifications will only be sent if task is assigned to someone other than yourself.</small>
-          </div>
-
-          <input type="hidden" name="priority" value="999" id="id_priority">
-          <input type="hidden" name="created_by" value="{{ request.user.id }}" id="id_created_by">
-          <input type="hidden" name="task_list" value="{{ task_list.id }}" id="id_task_list">
-          <input type="hidden" name="created_date" value="{{ created_date }}" id="id_created_date">
-          <p><input type="submit" name="add_task" value="Submit" class="btn btn-primary"></p>
-      </div>
-    </form>
     <hr />
   {% endif %}
 
@@ -74,6 +36,7 @@
             <th>Comm</th>
             <th>Del</th>
           </tr>
+
           {% for task in items %}
             <tr id="{{ task.id }}">
               <td>
@@ -112,6 +75,7 @@
         </table>
 
         <input type="submit" name="process_tasks" value="Process selection"  class="btn btn-sm btn-success">
+
         {% include 'todo/include/toggle_delete.html' %}
 
     </form>

todo/templates/todo/list_lists.html

@@ -1,6 +1,7 @@
 {% extends "todo/base.html" %}
 
 {% block title %}{{ list_title }} Todo Lists{% endblock %}
+
 {% block content %}
   <h1>Todo Lists</h1>
 

todo/templates/todo/task_detail.html

@@ -4,6 +4,9 @@
 
 {% block content %}
 
+  <button class="btn btn-primary" id="EditTaskButton" type="button"
+  data-toggle="collapse" data-target="#AddEditTask">Edit Task</button>
+
   <div class="card-deck">
     <div class="card col-sm-8">
       <div class="card-body">
@@ -18,11 +21,7 @@
       <ul class="list-group list-group-flush">
         <li class="list-group-item">
           <strong>Assigned to:</strong>
-                {% if task.assigned_to %}
+          {% if task.assigned_to %} {{ task.assigned_to.get_full_name }} {% else %} Anyone {% endif %}
-                  {{ task.assigned_to.get_full_name }}
-                {% else %}
-                  Anyone
-                {% endif %}
         </li>
         <li class="list-group-item">
           <strong>Reported by:</strong> {{ task.created_by.get_full_name }}
@@ -41,62 +40,22 @@
         </li>
       </ul>
     </div>
+
   </div>
 
-    <button id="slideToggle" class="mt-3 btn btn-primary">Edit task</button>
-
-    {% comment %} Needs form tag {% endcomment %}
   <div id="TaskEdit">
-      <h3>Edit Task</h3>
+    {# Task edit / new task form #}
-
+    {% include 'todo/include/task_edit.html' %}
-      <table>
-        <tr>
-          <td>Title:</td>
-          <td>{{ form.title }}
-          </td>
-        </tr>
-
-        <tr>
-          <td>List:</td>
-          <td>{{ form.task_list }}
-          </td>
-        </tr>
-
-        <tr>
-          <td>Due:</td>
-          <td>{{ form.due_date }}
-          </td>
-        </tr>
-
-        <tr>
-          <td>Assigned to:</td>
-          <td>{{ form.assigned_to }}
-          </td>
-        </tr>
-
-        <tr>
-          <td valign="top">Note:</td>
-          <td>{{ form.note }}
-          </td>
-        </tr>
-
-        <tr>
-          <td>Priority:</td>
-          <td>{{ form.priority }}
-          </td>
-        </tr>
-
-      </table>
-      <p><input type="submit" class="btn btn-primary" name="edit_task" value="Edit task"></p>
   </div>
 
-    <hr/>
-
   <h3>Add comment</h3>
-    <textarea name="comment-body"></textarea>
+  <form action="" method="post">
-    <p><input class="btn btn-primary" type="submit" value="Submit"></p>
+    {% csrf_token %}
-
+    <div class="form-group">
-  
+      <textarea class="form-control" name="comment-body" rows="3"></textarea>
+    </div>
+    <input class="btn btn-primary" type="submit" name="add_comment" value="Add Comment">
+  </form>
 
   <div class="task_comments">
     {% if comment_list %}
@@ -116,17 +75,3 @@
   </div>
 
 {% endblock %}
-
-{% block extra_js %}
-  <script type="text/javascript">
-    $(document).ready(function () {
-      // Initially hide the TaskEdit form
-      $('#TaskEdit').hide();
-
-      // toggle slide to show the Add Task form when link clicked
-      $('#slideToggle').click(function () {
-        $(this).siblings('#TaskEdit').slideToggle();
-      });
-    });
-  </script>
-{% endblock extra_js %}

todo/views.py

@@ -14,7 +14,7 @@ from django.template.loader import render_to_string
 from django.views.decorators.csrf import csrf_exempt
 
 from todo import settings
-from todo.forms import AddTaskListForm, AddItemForm, EditItemForm, AddExternalItemForm, SearchForm
+from todo.forms import AddTaskListForm, AddEditItemForm, AddExternalItemForm, SearchForm
 from todo.models import Item, TaskList, Comment
 from todo.utils import (
     toggle_done,
@@ -126,10 +126,11 @@ def list_detail(request, list_id=None, list_slug=None, view_completed=False):
     #  Add New Task Form
     # ######################
 
-    if request.POST.getlist('add_task'):
+    if request.POST.getlist('add_edit_task'):
-        form = AddItemForm(task_list, request.POST, initial={
+        form = AddEditItemForm(request.user, request.POST, initial={
             'assigned_to': request.user.id,
             'priority': 999,
+            'task_list': task_list
         })
 
         if form.is_valid():
@@ -144,9 +145,10 @@ def list_detail(request, list_id=None, list_slug=None, view_completed=False):
     else:
         # Don't allow adding new tasks on some views
         if list_slug not in ["mine", "recent-add", "recent-complete", ]:
-            form = AddItemForm(task_list=task_list, initial={
+            form = AddEditItemForm(request.user, initial={
                 'assigned_to': request.user.id,
                 'priority': 999,
+                'task_list': task_list
             })
 
     context = {
@@ -174,29 +176,29 @@ def task_detail(request, task_id: int) -> HttpResponse:
     if task.task_list.group not in request.user.groups.all() and not request.user.is_staff:
         raise PermissionDenied
 
-    if request.POST:
+    # Save submitted comments
-        form = EditItemForm(request.POST, instance=task)
+    if request.POST.get('add_comment'):
-
+        Comment.objects.create(
-        if form.is_valid():
-            form.save()
-
-            # Also save submitted comment, if non-empty
-            if request.POST['comment-body']:
-                c = Comment(
             author=request.user,
             task=task,
             body=request.POST['comment-body'],
         )
-                c.save()
 
         send_email_to_thread_participants(request, task)
-                messages.success(request, "Notification email sent to thread participants.")
+        messages.success(request, "Comment posted. Notification email sent to thread participants.")
 
+    # Save task edits
+    if request.POST.get('add_edit_task'):
+        form = AddEditItemForm(request.user, request.POST, instance=task, initial={'task_list': task.task_list})
+
+        if form.is_valid():
+            form.save()
             messages.success(request, "The task has been edited.")
 
             return redirect('todo:list_detail', list_id=task.task_list.id, list_slug=task.task_list.slug)
     else:
-        form = EditItemForm(instance=task)
+        form = AddEditItemForm(request.user, instance=task, initial={'task_list': task.task_list})
+
     if task.due_date:
         thedate = task.due_date
     else: