From 0f71ef9935aca1f99fc5a55469379776110429f4 Mon Sep 17 00:00:00 2001 From: Jared Vititoe Date: Sat, 11 Apr 2026 14:20:48 -0400 Subject: [PATCH] Fix ?priority=N filter from command palette not being applied The command palette 'P1 Critical Tickets' link uses ?priority=1, but getAllTickets only accepted priority_min/priority_max. Resolve the single ?priority=N param to both priority_min and priority_max so it acts as an exact priority match. Co-Authored-By: Claude Sonnet 4.6 --- controllers/DashboardController.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/controllers/DashboardController.php b/controllers/DashboardController.php index 394c040..03c9df2 100644 --- a/controllers/DashboardController.php +++ b/controllers/DashboardController.php @@ -131,9 +131,10 @@ class DashboardController { if ($closedFrom) $filters['closed_from'] = $closedFrom; if ($closedTo) $filters['closed_to'] = $closedTo; - // Validate priority filters - $priorityMin = $this->validatePriority($_GET['priority_min'] ?? null); - $priorityMax = $this->validatePriority($_GET['priority_max'] ?? null); + // Validate priority filters; ?priority=N sets exact match (min=max=N) + $prioritySingle = $this->validatePriority($_GET['priority'] ?? null); + $priorityMin = $prioritySingle ?? $this->validatePriority($_GET['priority_min'] ?? null); + $priorityMax = $prioritySingle ?? $this->validatePriority($_GET['priority_max'] ?? null); if ($priorityMin !== null) $filters['priority_min'] = $priorityMin; if ($priorityMax !== null) $filters['priority_max'] = $priorityMax;