repair: auto-fix system config types (#1168)

Signed-off-by: Varun Patil <[email protected]>
pulsejet · May 3, 2024 · 967c62c · 967c62c
1 parent f51c08d
commit 967c62c
Showing 1 changed file with 22 additions and 1 deletion.
diff --git a/lib/Migration/Repair.php b/lib/Migration/Repair.php
@@ -21,9 +21,13 @@ public function getName(): string
 
     public function run(IOutput $output): void
     {
-        // Add missing indices
         AddMissingIndices::run($output);
+        $this->configureBinExt($output);
+        $this->fixSystemConfigTypes($output);
+    }
 
+    public function configureBinExt(IOutput $output): void
+    {
         // kill any instances of go-vod and exiftool
         BinExt::pkill(BinExt::getName('go-vod'));
         BinExt::pkill(BinExt::getName('exiftool'));
@@ -49,4 +53,21 @@ public function run(IOutput $output): void
             $output->warning('ffmpeg binary could not be configured');
         }
     }
+
+    public function fixSystemConfigTypes(IOutput $output): void
+    {
+        // https://github.com/pulsejet/memories/issues/1168
+        $intKeys = [
+            'preview_max_x',
+            'preview_max_y',
+            'jpeg_quality',
+        ];
+        foreach ($intKeys as $key) {
+            $value = $this->config->getSystemValue($key, null);
+            if (null !== $value && !\is_int($value)) {
+                $output->info("Fixing system config value for {$key}");
+                $this->config->setSystemValue($key, (int) $value ?: 2048);
+            }
+        }
+    }
 }