[TASK] Drop Extbase usage for logs, resolves #362

ChangeLog

@@ -1,3 +1,11 @@
+2025-01-02 Francois Suter (Idéative)  <[email protected]>
+
+    * Drop Extbase usage for logs, resolves #362
+
+2024-12-31 Francois Suter (Idéative)  <[email protected]>
+
+    * Adapt to changes in connector services 6.0.0, resolves #351
+
 2024-12-27 Francois Suter (Idéative)  <[email protected]>
 
     * Add a delete reaction, resolves #361

Classes/Controller/LogAjaxController.php

@@ -37,7 +37,7 @@ class LogAjaxController
      * @return ResponseInterface
      * @throws \JsonException
      */
-    public function getAction(ServerRequestInterface $request, ?ResponseInterface $response): ResponseInterface
+    public function getAction(ServerRequestInterface $request): ResponseInterface
     {
         // Process query parameters
         $queryParameters = GeneralUtility::makeInstance(
@@ -48,7 +48,7 @@ public function getAction(ServerRequestInterface $request, ?ResponseInterface $r
         $logRepository = GeneralUtility::makeInstance(LogRepository::class);
 
         // Get the count of all the log entries
-        $totalEntries = $logRepository->findAll()->count();
+        $totalEntries = $logRepository->countAll();
         // Get the filtered entries and their count
         $logs = [];
         $logCount = 0;
@@ -61,13 +61,13 @@ public function getAction(ServerRequestInterface $request, ?ResponseInterface $r
             /** @var \Cobweb\ExternalImport\Domain\Model\Log $logEntry */
             foreach ($logEntries as $logEntry) {
                 $logs[] = [
-                    'status' => $logEntry->getStatus(),
-                    'crdate' => $logEntry->getCrdate()->format('U'),
-                    'username' => $logEntry->getCruserId() ? $logEntry->getCruserId()->getUserName() : '-',
-                    'configuration' => $logEntry->getConfiguration(),
-                    'context' => $logEntry->getContext(),
-                    'message' => $logEntry->getMessage(),
-                    'duration' => $logEntry->getDuration(),
+                    'status' => $logEntry['status'],
+                    'crdate' => $logEntry['crdate'],
+                    'username' => (string)$logEntry['cruser_id'] . ' TODO: retrieve user name',
+                    'configuration' => $logEntry['configuration'],
+                    'context' => $logEntry['context'],
+                    'message' => $logEntry['message'],
+                    'duration' => $logEntry['duration'],
                 ];
             }
         } catch (\Exception $e) {
@@ -85,9 +85,7 @@ public function getAction(ServerRequestInterface $request, ?ResponseInterface $r
             'recordsFiltered' => $logCount,
             'error' => $error,
         ];
-        if ($response === null) {
-            $response = GeneralUtility::makeInstance(JsonResponse::class);
-        }
+        $response = GeneralUtility::makeInstance(JsonResponse::class);
         $response->getBody()->write(json_encode($fullResponse, JSON_THROW_ON_ERROR));
         return $response;
     }