From 305eb0f66542f7aa8aff73dc884149e30334a0de Mon Sep 17 00:00:00 2001
From: DavidVadnais <david.vadnais@protonmail.com>
Date: Mon, 4 Aug 2025 17:24:51 +0000
Subject: [PATCH] switch assign user to use use update ticket not mass

---
 .../templates/helpdesk/ticket_desc_table.html | 30 +++++++++++++++----
 helpdesk/views/staff.py                       |  2 +-
 2 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/helpdesk/templates/helpdesk/ticket_desc_table.html b/helpdesk/templates/helpdesk/ticket_desc_table.html
index b0f4afa6..cef6dd73 100644
--- a/helpdesk/templates/helpdesk/ticket_desc_table.html
+++ b/helpdesk/templates/helpdesk/ticket_desc_table.html
@@ -77,16 +77,21 @@
                         <th class="table-active">{% trans "Assigned To" %}</th>
                         <td>
                             <!-- assignment drop down -->
-                            <form method="post" action="{% url 'helpdesk:mass_update' %}">
+                            <form method="post" action="{% url 'helpdesk:update' ticket.id %}">
                                 {% csrf_token %}
                                 <input type="hidden" name="ticket_id" value="{{ ticket.id }}">
-                                <input type="hidden" name="next" value="{% url 'helpdesk:view' ticket.id %}">
+                                <!-- Keep current queue and priority hidden so they're unchanged -->
+                                <input type="hidden" name="queue" value="{{ ticket.queue.id }}">
+                                <input type="hidden" name="priority" value="{{ ticket.priority }}">
+
+                                <!-- Summary / Title hidden to avoid validation error -->
+                                <input type="hidden" name="title" value="{{ ticket.title }}">
 
                                 <div class="d-flex align-items-center gap-2">
-                                    <select name="action" class="form-select form-select-sm">
-                                        <option disabled selected>Unassigned</option>
+                                    <select name="owner" class="form-select form-select-sm">
+                                        <option value="0" {% if not ticket.assigned_to %}selected{% endif %}>Unassigned</option>
                                         {% for user in assignable_users %}
-                                            <option value="assign_{{ user.id }}" {% if ticket.assigned_to and ticket.assigned_to.id == user.id %}selected{% endif %}>
+                                            <option value="{{ user.id }}" {% if ticket.assigned_to and ticket.assigned_to.id == user.id %}selected{% endif %}>
                                                 {{ user.get_full_name|default:user.username }}
                                             </option>
                                         {% endfor %}
@@ -104,6 +109,21 @@
                                     </a>
                                 </div>
                             </form>
+                            {% if form.errors %}
+                                <div class="alert alert-danger">
+                                    <strong>Form Errors:</strong>
+                                    <ul>
+                                    {% for field in form %}
+                                        {% for error in field.errors %}
+                                            <li>{{ field.label }}: {{ error }}</li>
+                                        {% endfor %}
+                                    {% endfor %}
+                                    {% for error in form.non_field_errors %}
+                                        <li>{{ error }}</li>
+                                    {% endfor %}
+                                    </ul>
+                                </div>
+                            {% endif %}
                         </td>
                         <th class="table-active">{% trans "Submitter E-Mail" %}</th>
                         <td>
diff --git a/helpdesk/views/staff.py b/helpdesk/views/staff.py
index 21d3efa5..711d76ae 100644
--- a/helpdesk/views/staff.py
+++ b/helpdesk/views/staff.py
@@ -479,7 +479,7 @@ def view_ticket(request, ticket_id):
 
     # Define users that the ticket can be assigned to
     assignable_users = User.objects.filter(is_active=True).order_by("username")
-    logger.debug("Assignable users:", assignable_users)
+    logger.debug("Assignable users: %s", assignable_users)
 
     return render(
         request,