add logging to cron

src/Cron/MeilisearchIndexCron.php

@@ -3,55 +3,52 @@
 namespace MummertMedia\ContaoMeilisearchBundle\Cron;
 
 use Contao\CoreBundle\Framework\ContaoFramework;
-use MummertMedia\ContaoMeilisearchBundle\Service\MeilisearchIndexService;
 use Symfony\Component\Process\Process;
 
 class MeilisearchIndexCron
 {
+    private string $logFile;
+
     public function __construct(
-        private readonly MeilisearchIndexService $indexService,
         private readonly ContaoFramework $framework,
         private readonly string $projectDir,
-    ) {}
+    ) {
+        $this->logFile = $this->projectDir . '/var/logs/meilisearch_bundle.log';
+    }
 
     public function __invoke(): void
     {
-        $start = microtime(true);
-
-        error_log('[MeilisearchCron] === START ===');
-
-        // Contao initialisieren
         $this->framework->initialize();
-        error_log('[MeilisearchCron] Contao framework initialized');
 
-        // 1) Contao Crawl
+        $this->log('=== CRON START ===');
-        $this->runConsole(
+
-            'contao:crawl',
+        // 1) Cleanup
-            'Contao crawl'
+        $this->runStep(
+            'meilisearch:files:cleanup',
+            'meilisearch:files:cleanup'
         );
 
-        // 2) Cleanup (24h Grace)
+        // 2) Contao Crawl
-        $this->runConsole(
+        $this->runStep(
-            'meilisearch:files:cleanup',
+            'contao:crawl',
-            'Meilisearch files cleanup'
+            'contao:crawl'
         );
 
         // 3) Meilisearch Index
-        try {
+        $this->runStep(
-            error_log('[MeilisearchCron] Meilisearch index started');
+            'meilisearch:index',
-            $this->indexService->run();
+            'meilisearch:index'
-            error_log('[MeilisearchCron] Meilisearch index finished');
+        );
-        } catch (\Throwable $e) {
-            error_log('[MeilisearchCron] ERROR during Meilisearch index: ' . $e->getMessage());
-        }
 
-        $duration = round(microtime(true) - $start, 2);
+        $this->log('=== CRON END ===');
-        error_log('[MeilisearchCron] === END (duration: ' . $duration . 's) ===');
     }
 
-    private function runConsole(string $command, string $label): void
+    /**
+     * Führt einen Console-Command aus und loggt sauber Start/Ende
+     */
+    private function runStep(string $command, string $label): void
     {
-        error_log('[MeilisearchCron] ' . $label . ' started');
+        $this->log($label . ' gestartet');
 
         $process = new Process([
             PHP_BINARY,
@@ -63,16 +60,30 @@ class MeilisearchIndexCron
         $process->run();
 
         if (!$process->isSuccessful()) {
-            error_log(
+            $this->log(
-                '[MeilisearchCron] ERROR in ' . $label . ': ' .
+                $label . ' FEHLGESCHLAGEN',
-                $process->getErrorOutput()
+                $process->getErrorOutput() ?: 'Unbekannter Fehler'
             );
-        } else {
+
-            $output = trim($process->getOutput());
+            // ❌ Abbruch – Folgeschritte NICHT ausführen
-            if ($output !== '') {
+            return;
-                error_log('[MeilisearchCron] ' . $label . ' output: ' . $output);
-            }
-            error_log('[MeilisearchCron] ' . $label . ' finished');
         }
+
+        $this->log($label . ' erfolgreich beendet');
+    }
+
+    /**
+     * Schreibt eine Logzeile mit Zeitstempel
+     */
+    private function log(string $message, string $details = ''): void
+    {
+        $line = sprintf(
+            "[%s] %s%s\n",
+            date('Y-m-d H:i:s'),
+            $message,
+            $details ? ' | ' . trim($details) : ''
+        );
+
+        file_put_contents($this->logFile, $line, FILE_APPEND);
     }
 }