From f6d18fc58d3f20dde3841e27c520aacc001e3767 Mon Sep 17 00:00:00 2001 From: Mathis Date: Tue, 30 Apr 2024 16:39:43 +0200 Subject: [PATCH] feat(services): simplify SQL query keys generation in mysql service Remove the condition to check 'id' while generating SQL query keys in the `mysql.service.ts`. Now, a map function is used directly resulting in cleaner and leaner code. Issue: #18 Signed-off-by: Mathis --- src/services/mysql.service.ts | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/services/mysql.service.ts b/src/services/mysql.service.ts index ae8596d..9443d01 100644 --- a/src/services/mysql.service.ts +++ b/src/services/mysql.service.ts @@ -66,12 +66,7 @@ class MysqlHandler { try { // @ts-ignore data.values.id ? delete data.values.id : null; - const _sqlQueryKeys = Object.keys(data.values).map((key: string) => { - if (key !== 'id') { - return `\'${key}\' = ?` - } - return ''; - }) + const _sqlQueryKeys = Object.keys(data.values).map((key: string) => `\'${key}\' = ?`) const values = Object.values(data.values).map((val)=>val) this.Logger.debug(`\n\n>-> Factorized ${_sqlQueryKeys.length} keys for a prepare Query.\n>-> Action: ${data.actionName}\n`) const sqlQueryKeys = _sqlQueryKeys.join(', ')