From 1532b089b99693168bde74b8b6ec27055fda4e0d Mon Sep 17 00:00:00 2001
From: Johan Haals <johan.haals@gmail.com>
Date: Mon, 26 Aug 2024 11:36:33 +0200
Subject: [PATCH] chore: simplify skip condition

Signed-off-by: Johan Haals <johan.haals@gmail.com>
---
 .../entrypoints/database/DatabaseManager.ts   | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/packages/backend-defaults/src/entrypoints/database/DatabaseManager.ts b/packages/backend-defaults/src/entrypoints/database/DatabaseManager.ts
index 42ba644823..b3e69f0ae2 100644
--- a/packages/backend-defaults/src/entrypoints/database/DatabaseManager.ts
+++ b/packages/backend-defaults/src/entrypoints/database/DatabaseManager.ts
@@ -87,18 +87,13 @@ export class DatabaseManagerImpl implements LegacyRootDatabaseService {
     }
     const getClient = () => this.getDatabase(pluginId, connector, deps);
 
-    let skip = false;
-    // class options take precedence over config
-    if (this.options?.migrations?.skip !== undefined) {
-      skip = this.options.migrations.skip;
-    } else {
-      skip =
-        this.config.getOptionalBoolean(
-          `backend.database.plugin.${pluginId}.skipMigrations`,
-        ) ??
-        this.config.getOptionalBoolean('backend.database.skipMigrations') ??
-        false;
-    }
+    const skip =
+      this.options?.migrations?.skip ??
+      this.config.getOptionalBoolean(
+        `backend.database.plugin.${pluginId}.skipMigrations`,
+      ) ??
+      this.config.getOptionalBoolean('backend.database.skipMigrations') ??
+      false;
 
     return { getClient, migrations: { skip } };
   }