Fix ticket quick actions to use model constants for status/priority

- Replace hardcoded status array with Ticket::ALL_STATUSES
- Replace hardcoded priority array with Ticket::ALL_PRIORITIES
- Update validation rules to use Ticket::getStatusValidationRule()
- Update validation rules to use Ticket::getPriorityValidationRule()
- Ensures quick action dropdowns show all valid options including 'new' status

Author: o-psi

app/Livewire/Tickets/TicketShow.php

@@ -85,15 +85,18 @@ class TicketShow extends Component
         'confirmed-start-timer' => 'handleConfirmedStart',
     ];
 
-    protected $rules = [
-        'comment' => 'required|min:5|max:5000',
-        'attachments.*' => 'file|max:10240|mimes:pdf,doc,docx,xls,xlsx,txt,jpg,jpeg,png,gif',
-        'status' => 'required|in:Open,In Progress,On Hold,Resolved,Closed',
-        'priority' => 'required|in:Low,Medium,High,Critical',
-        'assignedTo' => 'nullable|exists:users,id',
-        'timeSpent' => 'nullable|numeric|min:0.1|max:999',
-        'timeDescription' => 'required_with:timeSpent|string|max:500',
-    ];
+    protected function rules()
+    {
+        return [
+            'comment' => 'required|min:5|max:5000',
+            'attachments.*' => 'file|max:10240|mimes:pdf,doc,docx,xls,xlsx,txt,jpg,jpeg,png,gif',
+            'status' => 'required|'.Ticket::getStatusValidationRule(),
+            'priority' => 'required|'.Ticket::getPriorityValidationRule(),
+            'assignedTo' => 'nullable|exists:users,id',
+            'timeSpent' => 'nullable|numeric|min:0.1|max:999',
+            'timeDescription' => 'required_with:timeSpent|string|max:500',
+        ];
+    }
 
     protected $messages = [
         'comment.required' => 'Please enter a comment.',
@@ -219,7 +222,7 @@ public function updateStatus()
     {
         try {
             $this->validate([
-                'newStatus' => 'required|in:Open,In Progress,On Hold,Resolved,Closed',
+                'newStatus' => 'required|'.Ticket::getStatusValidationRule(),
                 'statusChangeReason' => 'required|min:10|max:500',
             ]);
 
@@ -261,7 +264,7 @@ public function updateStatus()
 
     public function updatePriority()
     {
-        $this->validate(['priority' => 'required|in:low,medium,high,urgent,critical']);
+        $this->validate(['priority' => 'required|'.Ticket::getPriorityValidationRule()]);
 
         $oldPriority = $this->ticket->priority;
         $this->ticket->priority = $this->priority;
@@ -321,15 +324,15 @@ public function toggleWatch()
         if ($watcher) {
             $watcher->delete();
             $this->ticket->load('watchers.user');
-            
+
             Flux::toast(
                 text: 'You are no longer watching this ticket',
                 variant: 'info'
             );
         } else {
             $this->ticket->watchers()->create(['user_id' => Auth::id()]);
             $this->ticket->load('watchers.user');
-            
+
             Flux::toast(
                 text: 'You are now watching this ticket',
                 variant: 'success'
@@ -347,7 +350,7 @@ public function deleteComment($commentId)
         if ($comment && $comment->created_at->diffInMinutes(now()) < 30) {
             $comment->delete();
             $this->ticket->load('comments.author', 'comments.attachments');
-            
+
             Flux::toast(
                 text: 'Comment deleted successfully',
                 variant: 'success'
@@ -392,7 +395,7 @@ public function updateComment()
             $comment->update(['content' => $this->editingCommentText]);
             $this->reset(['editingCommentId', 'editingCommentText']);
             $this->ticket->load('comments.author', 'comments.attachments');
-            
+
             Flux::toast(
                 text: 'Comment updated successfully',
                 variant: 'success'
@@ -426,7 +429,7 @@ public function cloneTicket()
     public function archiveTicket()
     {
         $this->ticket->update(['archived_at' => now()]);
-        
+
         Flux::toast(
             text: 'Ticket archived successfully',
             variant: 'success'
@@ -444,7 +447,7 @@ public function deleteTimeEntry($timeEntryId)
         if ($entry) {
             $entry->delete();
             $this->ticket->load('timeLogs.user');
-            
+
             Flux::toast(
                 text: 'Time entry deleted successfully',
                 variant: 'success'
@@ -590,7 +593,7 @@ public function saveDraft()
                 now()->addDays(1)
             );
             $this->draftSaved = true;
-            
+
             // Reset the draft saved indicator after 2 seconds
             $this->dispatch('draftSavedIndicator');
         }
@@ -641,8 +644,8 @@ public function render()
         return view('livewire.tickets.ticket-show', [
             'technicians' => $technicians,
             'isWatching' => $isWatching,
-            'statuses' => ['open', 'in_progress', 'pending', 'resolved', 'closed'],
-            'priorities' => ['low', 'medium', 'high', 'urgent', 'critical'],
+            'statuses' => Ticket::ALL_STATUSES,
+            'priorities' => Ticket::ALL_PRIORITIES,
         ]);
     }
 }