fix(cdr): implement GET /cdr/{id} retrieval and accept linkedid (#997)

GetRecordAction was a TODO stub returning empty data for any valid CDR ID.
Implements lookup by numeric primary key or linkedid (mikopbx-*), formats
response via DataStructure::createFromModel(), and returns 404 when not found.

Also aligns the getRecord id pattern with delete (^([0-9]+|mikopbx-.+)$) so
the same identifier works for both endpoints — the prior numeric-only pattern
was inconsistent with DeleteRecordAction's behavior.

Author: Alexey Portnov

src/PBXCoreREST/Controllers/Cdr/RestController.php

@@ -66,8 +66,8 @@
     // 'mikopbx-' generates mikopbx-[^/:]+  (matches linkedid like "mikopbx-1760784793.4627")
     //
     // IMPORTANT: Individual methods further restrict ID format via ApiParameterRef pattern:
-    // - getRecord, playback, download: numeric only (pattern: '^[0-9]+$')
-    // - delete: numeric OR linkedid (pattern: '^([0-9]+|mikopbx-.+)$')
+    // - getRecord, delete: numeric OR linkedid (pattern: '^([0-9]+|mikopbx-.+)$')
+    // - playback, download: numeric only (pattern: '^[0-9]+$')
     //
     // SECURITY NOTE: ResourceSecurity removed from class level because this resource has mixed security:
     // - getList/getRecord/delete: require Bearer token (added at method level)
@@ -129,7 +129,7 @@ public function getList(): void
         description: 'rest_cdr_GetRecordDesc',
         operationId: 'getCdrById'
     )]
-    #[ApiParameterRef('id', dataStructure: CommonDataStructure::class, pattern: '^[0-9]+$', example: '12345')]
+    #[ApiParameterRef('id', dataStructure: CommonDataStructure::class, pattern: '^([0-9]+|mikopbx-.+)$', example: '12345')]
     #[ApiResponse(200, 'rest_response_200_get')]
     #[ApiResponse(401, 'rest_response_401_unauthorized', 'PBXApiResult')]
     #[ApiResponse(403, 'rest_response_403_forbidden', 'PBXApiResult')]

src/PBXCoreREST/Lib/Cdr/GetRecordAction.php

@@ -19,36 +19,86 @@
 
 namespace MikoPBX\PBXCoreREST\Lib\Cdr;
 
+use MikoPBX\Common\Models\CallDetailRecords;
 use MikoPBX\PBXCoreREST\Lib\PBXApiResult;
+use Throwable;
 
 /**
  * Get single CDR record action
  *
+ * Supports two lookup modes based on ID format (mirrors DeleteRecordAction):
+ * - numeric ID: lookup by primary key
+ * - linkedid (mikopbx-*): returns the originating record of the conversation
+ *   (earliest by start time). Use GET /cdr?linkedid=... to fetch every segment
+ *   of a transferred call.
+ *
  * @package MikoPBX\PBXCoreREST\Lib\Cdr
  */
 class GetRecordAction
 {
     /**
-     * Get single CDR record
+     * Get single CDR record by numeric ID or linkedid.
      *
-     * @param string|null $id Record ID
+     * @param string|null $id Record ID (numeric) or linkedid (mikopbx-*)
      * @return PBXApiResult
      */
     public static function main(?string $id = null): PBXApiResult
     {
         $res = new PBXApiResult();
         $res->processor = __METHOD__;
 
+        // ============ PHASE 1: VALIDATION ============
         if (empty($id)) {
             $res->messages['error'][] = 'Record ID is required';
+            $res->httpCode = 400;
+            return $res;
+        }
+
+        // ============ PHASE 2: DETERMINE ID TYPE ============
+        $isLinkedId = str_starts_with($id, 'mikopbx-');
+        // Reject decimal numbers (12.34) — only accept integers.
+        $isNumericId = is_numeric($id) && (string)(int)$id === $id;
+
+        if (!$isLinkedId && !$isNumericId) {
+            $res->messages['error'][] = 'Invalid ID format. Expected: integer ID or linkedid (mikopbx-*)';
+            $res->httpCode = 400;
+            return $res;
+        }
+
+        // ============ PHASE 3: FETCH RECORD ============
+        try {
+            if ($isLinkedId) {
+                // Originating record of the conversation — earliest segment.
+                // Full conversation is available via GET /cdr?linkedid=...
+                $record = CallDetailRecords::findFirst([
+                    'conditions' => 'linkedid = :linkedid:',
+                    'bind' => ['linkedid' => $id],
+                    'order' => 'start ASC',
+                ]);
+            } else {
+                $record = CallDetailRecords::findFirst([
+                    'conditions' => 'id = :id:',
+                    'bind' => ['id' => (int)$id],
+                ]);
+            }
+        } catch (Throwable $e) {
+            $res->messages['error'][] = 'Failed to query CDR record: ' . $e->getMessage();
+            $res->httpCode = 500;
+            return $res;
+        }
+
+        if ($record === null) {
+            $res->messages['error'][] = 'CDR record not found';
+            $res->httpCode = 404;
             return $res;
         }
 
-        // TODO: Implement single CDR record retrieval
-        // This is a placeholder for future implementation
-        $res->data = [];
+        // ============ PHASE 4: FORMAT RESPONSE ============
+        // createFromModel() adds playback_url/download_url and applies schema typing.
+        $res->data = DataStructure::createFromModel($record);
         $res->success = true;
+        $res->httpCode = 200;
 
         return $res;
     }
-}
+}