fix: updated isLoggerEnabled to setLogger

JS-AK · Dec 10, 2024 · cf8455b · cf8455b
1 parent e6c0210
commit cf8455b
Show file tree

Hide file tree

Showing 12 changed files with 26 additions and 100 deletions.
diff --git a/src/lib/mysql/model/materialized-view.ts b/src/lib/mysql/model/materialized-view.ts
@@ -70,16 +70,10 @@ export class BaseMaterializedView {
 		this.#logger = logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/mysql/model/model.ts b/src/lib/mysql/model/model.ts
@@ -58,16 +58,10 @@ export class BaseModel<const T extends readonly string[] = readonly string[]> {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/mysql/model/sequence.ts b/src/lib/mysql/model/sequence.ts
@@ -36,16 +36,10 @@ export class BaseSequence {
 		this.#logger = logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/mysql/model/table.ts b/src/lib/mysql/model/table.ts
@@ -93,16 +93,10 @@ export class BaseTable<const T extends readonly string[] = readonly string[]> {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/mysql/model/view.ts b/src/lib/mysql/model/view.ts
@@ -70,16 +70,10 @@ export class BaseView {
 		this.#logger = logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/mysql/repository-manager/repository-manager.ts b/src/lib/mysql/repository-manager/repository-manager.ts
@@ -5,6 +5,7 @@ import { QueryBuilderFactory } from "../query-builder/query-builder-factory.js";
 import { TransactionManager } from "../transaction-manager/transaction-manager.js";
 
 import * as SharedTypes from "../../../shared-types/index.js";
+import * as Types from "../model/types.js";
 
 type Model =
 	| Model.BaseMaterializedView
@@ -13,14 +14,6 @@ type Model =
 	| Model.BaseTable
 	| Model.BaseView;
 
-type Config = {
-	database: string;
-	host: string;
-	password: string;
-	port: number;
-	user: string;
-};
-
 /**
  * RepositoryManager class.
  * It is a class for managing MYSQL repositories, query builder factory and transaction manager.
@@ -46,7 +39,7 @@ export class RepositoryManager<const T extends Record<string, { model: Model; }>
 	constructor(
 		repository: T,
 		options: {
-			config: Config;
+			config: Types.TDBCreds;
 			logger: SharedTypes.TLogger;
 			isLoggerEnabled?: boolean;
 		},
@@ -67,7 +60,7 @@ export class RepositoryManager<const T extends Record<string, { model: Model; }>
 					continue;
 				}
 
-				r.model.isLoggerEnabled = true;
+				r.model.setLogger(this.#logger);
 			}
 		}
 	}

diff --git a/src/lib/pg/model/materialized-view.ts b/src/lib/pg/model/materialized-view.ts
@@ -75,16 +75,10 @@ export class BaseMaterializedView {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/pg/model/model.ts b/src/lib/pg/model/model.ts
@@ -54,16 +54,10 @@ export class BaseModel<const T extends readonly string[] = readonly string[]> {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/pg/model/sequence.ts b/src/lib/pg/model/sequence.ts
@@ -41,16 +41,10 @@ export class BaseSequence {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/pg/model/table.ts b/src/lib/pg/model/table.ts
@@ -89,16 +89,10 @@ export class BaseTable<const T extends readonly string[] = readonly string[]> {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/pg/model/view.ts b/src/lib/pg/model/view.ts
@@ -75,16 +75,10 @@ export class BaseView {
 		this.#logger = preparedOptions.logger;
 	}
 
-	set isLoggerEnabled(value: boolean) {
-		const prev = this.#isLoggerEnabled;
-
-		if (prev === value) {
-			return;
-		}
-
+	setLogger(logger: SharedTypes.TLogger) {
 		const preparedOptions = setLoggerAndExecutor(
 			this.pool,
-			{ isLoggerEnabled: value, logger: this.#logger },
+			{ isLoggerEnabled: true, logger },
 		);
 
 		this.#executeSql = preparedOptions.executeSql;

diff --git a/src/lib/pg/repository-manager/repository-manager.ts b/src/lib/pg/repository-manager/repository-manager.ts
@@ -4,6 +4,7 @@ import { QueryBuilderFactory } from "../query-builder/query-builder-factory.js";
 import { TransactionManager } from "../transaction-manager/transaction-manager.js";
 
 import * as SharedTypes from "../../../shared-types/index.js";
+import * as Types from "../model/types.js";
 
 type Model =
 	| Model.BaseMaterializedView
@@ -12,14 +13,6 @@ type Model =
 	| Model.BaseTable
 	| Model.BaseView;
 
-type Config = {
-	database: string;
-	host: string;
-	password: string;
-	port: number;
-	user: string;
-};
-
 /**
  * RepositoryManager class.
  * It is a class for managing PG repositories, query builder factory and transaction manager.
@@ -45,7 +38,7 @@ export class RepositoryManager<const T extends Record<string, { model: Model; }>
 	constructor(
 		repository: T,
 		options: {
-			config: Config;
+			config: Types.TDBCreds;
 			logger: SharedTypes.TLogger;
 			isLoggerEnabled?: boolean;
 		},
@@ -68,7 +61,7 @@ export class RepositoryManager<const T extends Record<string, { model: Model; }>
 					continue;
 				}
 
-				r.model.isLoggerEnabled = true;
+				r.model.setLogger(this.#logger);
 			}
 		}
 	}