feat(tickets): getMyClosedTicketsForRange for period close-lists

app/Domain/Tickets/Services/Tickets.php

@@ -2418,12 +2418,43 @@ public function getAllDoneUserTickets(?int $userId = null, ?int $project = null)
      * then re-completed the same day is included once. Built on the general
      * getStatusChangeEvents primitive, so throughput/burndown and strategy
      * reporting can reuse the same query.
+     *
+     * Thin wrapper over {@see getMyClosedTicketsForRange} with from == to.
      */
     #[RequiresPermission(TicketsPermissions::VIEW)]
     public function getMyClosedTicketsForDate(?int $userId = null, ?string $date = null): array
     {
         $date = $date ?: date('Y-m-d');
 
+        return $this->getMyClosedTicketsForRange($userId, $date, $date);
+    }
+
+    /**
+     * @api
+     *
+     * Range form of {@see getMyClosedTicketsForDate}: the user's tasks marked
+     * DONE anywhere within [$from, $to] (inclusive, dates 'Y-m-d'), each
+     * annotated with `dateClosed` (the completion timestamp). Powers the mobile
+     * Progress "Closed this week / this month" sections — completed arcs are
+     * keyed on close-date within the period, independent of how recently the
+     * project was otherwise touched, so finished work never disappears.
+     *
+     * Same "closed" definition as the single-date form: the ticket is currently
+     * DONE and its status was changed to that DONE status within the range. A
+     * ticket completed more than once in the range is included once, keyed to
+     * its latest completion (events are newest-first). Defaults to today when
+     * either bound is omitted.
+     */
+    #[RequiresPermission(TicketsPermissions::VIEW)]
+    public function getMyClosedTicketsForRange(?int $userId = null, ?string $from = null, ?string $to = null): array
+    {
+        $from = $from ?: date('Y-m-d');
+        $to = $to ?: date('Y-m-d');
+        // Tolerate a reversed range rather than returning nothing.
+        if ($from > $to) {
+            [$from, $to] = [$to, $from];
+        }
+
         // Candidates: the user's currently-DONE tickets (statusType resolved).
         $doneTickets = $this->getAllDoneUserTickets($userId);
         if (empty($doneTickets)) {
@@ -2435,7 +2466,7 @@ public function getMyClosedTicketsForDate(?int $userId = null, ?string $date = n
             $byId[$ticket['id']] = $ticket;
         }
 
-        $events = $this->ticketRepository->getStatusChangeEvents(array_keys($byId), $date, $date);
+        $events = $this->ticketRepository->getStatusChangeEvents(array_keys($byId), $from, $to);
 
         $closed = [];
         foreach ($events as $event) {
@@ -2446,8 +2477,8 @@ public function getMyClosedTicketsForDate(?int $userId = null, ?string $date = n
             }
 
             // Only count changes INTO the ticket's current (DONE) status — it
-            // was actually marked done that day, not merely touched. Events are
-            // newest-first, so the first match keeps the latest completion time.
+            // was actually marked done in the range, not merely touched. Events
+            // are newest-first, so the first match keeps the latest completion.
             if ((string) $event['changeValue'] === (string) $ticket['status']) {
                 $ticket['dateClosed'] = $event['dateModified'];
                 $closed[$ticketId] = $ticket;