From dccec2d5f599c69b0ae544a0d16da997dd581af1 Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Sat, 9 Dec 2023 15:04:30 -0500
Subject: [PATCH] Removed actions column from tickets as edit ticket and Delete
 should be done within the ticket itself, however you can still change
 priotity and assigned to within the list view. This makes a tremendous impact
 on performance as ticket details wont need loaded for each row Soon we will
 impelment bulk actions

---
 tickets.php | 51 ++++++++++-----------------------------------------
 1 file changed, 10 insertions(+), 41 deletions(-)

diff --git a/tickets.php b/tickets.php
index 1d471b30..e369638e 100644
--- a/tickets.php
+++ b/tickets.php
@@ -265,7 +265,6 @@ $user_active_assigned_tickets = intval($row['total_tickets_assigned']);
                            href="?<?php echo $url_query_strings_sort; ?>&sort=ticket_created_at&order=<?php echo $disp; ?>">Created</a>
                     </th>
 
-                    <th class="text-center">Action</th>
                 </tr>
                 </thead>
                 <tbody>
@@ -276,7 +275,6 @@ $user_active_assigned_tickets = intval($row['total_tickets_assigned']);
                     $ticket_prefix = nullable_htmlentities($row['ticket_prefix']);
                     $ticket_number = intval($row['ticket_number']);
                     $ticket_subject = nullable_htmlentities($row['ticket_subject']);
-                    $ticket_details = nullable_htmlentities($row['ticket_details']);
                     $ticket_priority = nullable_htmlentities($row['ticket_priority']);
                     $ticket_status = nullable_htmlentities($row['ticket_status']);
                     $ticket_vendor_ticket_number = nullable_htmlentities($row['ticket_vendor_ticket_number']);
@@ -356,7 +354,7 @@ $user_active_assigned_tickets = intval($row['total_tickets_assigned']);
                         <td>
                             <a href="client_tickets.php?client_id=<?php echo $client_id; ?>"><strong><?php echo $client_name; ?></strong></a>
 
-                            <div class="mt-1"><a href="#" data-toggle="modal" data-target="#editTicketContactModal<?php echo $ticket_id; ?>"><?php echo $contact_display; ?></a></div>
+                            <div class="mt-1"><?php echo $contact_display; ?></div>
                         </td>
                         <td><a href="#" data-toggle="modal" data-target="#editTicketPriorityModal<?php echo $ticket_id; ?>"><span class='p-2 badge badge-pill badge-<?php echo $ticket_priority_color; ?>'><?php echo $ticket_priority; ?></span></a></td>
                         <td><span class='p-2 badge badge-pill badge-<?php echo $ticket_status_color; ?>'><?php echo $ticket_status; ?></span></td>
@@ -367,46 +365,19 @@ $user_active_assigned_tickets = intval($row['total_tickets_assigned']);
                             <br>
                             <small class="text-secondary"><?php echo $ticket_created_at; ?></small>
                         </td>
-                        <td>
-                            <?php if ($ticket_status !== "Closed") { ?>
-                                <div class="dropdown dropleft text-center">
-                                    <button class="btn btn-secondary btn-sm" type="button" data-toggle="dropdown">
-                                        <i class="fas fa-ellipsis-h"></i>
-                                    </button>
-                                    <div class="dropdown-menu">
-                                        <a class="dropdown-item" href="#" data-toggle="modal" data-target="#editTicketModal<?php echo $ticket_id; ?>">
-                                            <i class="fas fa-fw fa-edit mr-2"></i>Edit
-                                        </a>
-                                        <?php if ($session_user_role == 3) { ?>
-                                            <div class="dropdown-divider"></div>
-                                            <a class="dropdown-item text-danger text-bold confirm-link" href="post.php?delete_ticket=<?php echo $ticket_id; ?>">
-                                                <i class="fas fa-fw fa-trash mr-2"></i>Delete
-                                            </a>
-                                        <?php } ?>
-                                    </div>
-                                </div>
-                            <?php }
-
-                            if ($ticket_status !== "Closed") {
-                                // Temp performance boost for closed tickets, until we move to dynamic modals
-
-                                require "ticket_edit_modal.php";
-
-                                require "ticket_assign_modal.php";
-
-                                require "ticket_edit_priority_modal.php";
-
-                                require "ticket_edit_contact_modal.php";
-
-                            }
-
-
-                            ?>
-                        </td>
                     </tr>
 
                     <?php
 
+                    if ($ticket_status !== "Closed") {
+                        // Temp performance boost for closed tickets, until we move to dynamic modals
+
+                        require "ticket_assign_modal.php";
+
+                        require "ticket_edit_priority_modal.php";
+
+                    }
+
                 }
 
                 ?>
@@ -425,5 +396,3 @@ require_once "ticket_add_modal.php";
 require_once "footer.php";
 
 ?>
-
-<script src="js/ticket_add_remove_watchers.js"></script>