fix: address Copilot review feedback on moveTicket + sprint filter

- moveTicket: pass null (not '') to the nullable integer columns
  sprint/dependingTicketId/milestoneid so Postgres accepts the clear;
  check the child patch() return value and short-circuit on failure
  so a milestone can't end up half-moved; drop the duplicate
  ticket_updated dispatch since patch() already fires it on success.
- Sprint filter (both call sites in Tickets repo): replace the
  is_numeric guard with per-element ctype_digit validation so
  comma-separated multi-sprint filters like "1,2,3" work again,
  while 'backlog' / empty / non-digit values still fall through to
  the dedicated backlog branch.

Addresses review comments on #3360.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: marcelfolaron

app/Domain/Tickets/Repositories/Tickets.php

@@ -561,9 +561,15 @@ public function getAllBySearchCriteria(array $searchCriteria, string $sort = 'st
             });
         }
 
-        if (isset($searchCriteria['sprint']) && is_numeric($searchCriteria['sprint']) && $searchCriteria['sprint'] > 0) {
-            $sprintIds = explode(',', $searchCriteria['sprint']);
-            $query->whereIn('zp_tickets.sprint', $sprintIds);
+        if (isset($searchCriteria['sprint']) && $searchCriteria['sprint'] !== '' && $searchCriteria['sprint'] !== 'backlog') {
+            $sprintIds = array_values(array_filter(
+                array_map('trim', explode(',', (string) $searchCriteria['sprint'])),
+                static fn ($token) => ctype_digit($token) && (int) $token > 0
+            ));
+
+            if ($sprintIds !== []) {
+                $query->whereIn('zp_tickets.sprint', array_map('intval', $sprintIds));
+            }
         }
 
         if (isset($searchCriteria['sprint']) && $searchCriteria['sprint'] === 'backlog') {
@@ -1224,12 +1230,19 @@ public function getAllMilestones(array $searchCriteria, string $sort = 'standard
             });
         }
 
-        if (isset($searchCriteria['sprint']) && is_numeric($searchCriteria['sprint']) && $searchCriteria['sprint'] > 0) {
-            $sprintIds = explode(',', $searchCriteria['sprint']);
-            $query->where(function ($q) use ($sprintIds) {
-                $q->whereIn('zp_tickets.sprint', $sprintIds)
-                    ->orWhere('zp_tickets.type', 'milestone');
-            });
+        if (isset($searchCriteria['sprint']) && $searchCriteria['sprint'] !== '' && $searchCriteria['sprint'] !== 'backlog') {
+            $sprintIds = array_values(array_filter(
+                array_map('trim', explode(',', (string) $searchCriteria['sprint'])),
+                static fn ($token) => ctype_digit($token) && (int) $token > 0
+            ));
+
+            if ($sprintIds !== []) {
+                $intSprintIds = array_map('intval', $sprintIds);
+                $query->where(function ($q) use ($intSprintIds) {
+                    $q->whereIn('zp_tickets.sprint', $intSprintIds)
+                        ->orWhere('zp_tickets.type', 'milestone');
+                });
+            }
         }
 
         if (isset($searchCriteria['sprint']) && $searchCriteria['sprint'] === 'backlog') {

app/Domain/Tickets/Services/Tickets.php

@@ -2235,17 +2235,22 @@ public function moveTicket(int $id, int $projectId): bool
         if ($ticket->type == 'milestone') {
             $milestoneTickets = $this->getAll(['milestone' => $ticket->id]);
             foreach ($milestoneTickets as $childTicket) {
-                $this->patch($childTicket['id'], ['projectId' => $projectId, 'sprint' => '']);
+                $childMoved = $this->patch($childTicket['id'], [
+                    'projectId' => $projectId,
+                    'sprint' => null,
+                ]);
+
+                if (! $childMoved) {
+                    return false;
+                }
             }
         }
 
-        self::dispatchEvent('ticket_updated');
-
         return $this->patch($ticket->id, [
             'projectId' => $projectId,
-            'sprint' => '',
-            'dependingTicketId' => '',
-            'milestoneid' => '',
+            'sprint' => null,
+            'dependingTicketId' => null,
+            'milestoneid' => null,
         ]);
     }