From 05d979b3d8ef6a3572f10e9511ab7d9a35695bc1 Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Fri, 5 Apr 2024 14:19:34 -0400
Subject: [PATCH] Do not allow editing a closed project

---
 project_details.php |  7 ++++++-
 projects.php        | 13 ++++++++-----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/project_details.php b/project_details.php
index 1e2d09d3..2cfb0fe2 100644
--- a/project_details.php
+++ b/project_details.php
@@ -35,8 +35,10 @@ if (isset($_GET['project_id'])) {
 
     if($project_completed_at) {
         $project_status_display = "<span class='badge badge-pill badge-dark ml-2'>Closed</span>";
+        $project_completed_date_display = "<div class='text-primary text-bold'><small><i class='fa fa-fw fa-door-closed mr-2'></i>" . date('Y-m-d', strtotime($project_completed_at)) . "</small></div>";
     } else {
         $project_status_display = "<span class='badge badge-pill badge-primary ml-2'>Open</span>";
+        $project_completed_date_display = "";
     }
 
     // Get Tickets
@@ -106,6 +108,7 @@ if (isset($_GET['project_id'])) {
                 <div class="media-body">
                     <h3 class="mb-0"><?php echo $client_name; ?></h3>
                     <div><small class='text-secondary'><i class='fa fa-fw fa-clock mr-2'></i><?php echo $project_due; ?></small></div>
+                    <?php echo $project_completed_date_display; ?>
                 </div>
             </div>
         </div>
@@ -137,14 +140,16 @@ if (isset($_GET['project_id'])) {
                         <i class="fas fa-fw fa-ellipsis-v"></i>
                     </button>
                     <div class="dropdown-menu">
+                        <?php if(empty($project_completed_at)) { ?>
                         <a class="dropdown-item" href="#" data-toggle="modal" data-target="#editProjectModal<?php echo $project_id; ?>">
                             <i class="fas fa-fw fa-edit mr-2"></i>Edit
                         </a>
                         <a class="dropdown-item" href="#" data-toggle="modal" data-target="#addProjectTicketModal">
                             <i class="fas fa-fw fa-life-ring mr-2"></i>Add Ticket
                         </a>
+                        <div class="dropdown-divider"></div>
+                        <?php } ?>
                         <?php if ($session_user_role == 3) { ?>
-                            <div class="dropdown-divider"></div>
                             <a class="dropdown-item text-danger text-bold confirm-link" href="post.php?archive_project=<?php echo $project_id; ?>">
                                 <i class="fas fa-fw fa-archive mr-2"></i>Archive
                             </a>
diff --git a/projects.php b/projects.php
index 83366423..50ea65ed 100644
--- a/projects.php
+++ b/projects.php
@@ -127,8 +127,10 @@ $num_rows = mysqli_fetch_row(mysqli_query($mysqli, "SELECT FOUND_ROWS()"));
                         $project_name = nullable_htmlentities($row['project_name']);
                         $project_description = nullable_htmlentities($row['project_description']);
                         $project_due = nullable_htmlentities($row['project_due']);
-                        $project_completed_at = date("Y-m-d", strtotime($row['project_completed_at']));
-                        $project_created_at = date("Y-m-d", strtotime($row['project_created_at']));
+                        $project_completed_at = nullable_htmlentities($row['project_completed_at']);
+                        $project_completed_at_display = date("Y-m-d", strtotime($project_completed_at));
+                        $project_created_at = nullable_htmlentities($row['project_created_at']);
+                        $project_created_at_display = date("Y-m-d", strtotime($project_created_at));
                         $project_updated_at = nullable_htmlentities($row['project_updated_at']);
 
                         $client_id = intval($row['client_id']);
@@ -200,25 +202,26 @@ $num_rows = mysqli_fetch_row(mysqli_query($mysqli, "SELECT FOUND_ROWS()"));
                             </td>
                             <td><?php echo $project_due; ?></td>
                             <?php if ($status == 1) { ?>
-                            <td><?php echo $project_completed_at; ?></td>
+                            <td><?php echo $project_completed_at_display; ?></td>
                             <?php } ?>
                             <td>
                                 <a href="client_tickets.php?client_id=<?php echo $client_id; ?>">
                                     <?php echo $client_name; ?>
                                 </a>
                             </td>
-                            <td><?php echo $project_created_at; ?></td>
+                            <td><?php echo $project_created_at_display; ?></td>
                             <td>
                                 <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">
-                                        
+                                        <?php if(empty($project_completed_at)) { ?>
                                         <a class="dropdown-item" href="#" data-toggle="modal" data-target="#editProjectModal<?php echo $project_id; ?>">
                                             <i class="fas fa-fw fa-edit mr-2"></i>Edit
                                         </a>
                                         <div class="dropdown-divider"></div>
+                                        <?php } ?>
                                         <a class="dropdown-item text-danger confirm-link" href="post.php?delete_project=<?php echo $project_id; ?>">
                                             <i class="fas fa-fw fa-archive mr-2"></i>Delete
                                         </a>