ismx-nexo-node-app 0.4.18 → 0.4.19

package/dist/js/repository/RepositoryDatabasePostgres.js

@@ -152,11 +152,15 @@ class RepositoryDatabasePostgres extends RepositoryDatabase_1.default {
                 throw new Error(`table ${tableName} does not exist`);
             let table = this.tables[tableName];
             let schema = table[0].schema;
-            let columns = PostgresUtils_1.default.columns(table);
-            let params = columns.map((column, index) => `\$${index + 2}`).join(",");
-            let values = [id, ...columns.map((column) => object[PostgresUtils_1.default.stringToCamel(column)])];
-            let query = `UPDATE ${schema}.${tableName} SET (${columns.join(",")}) = (${params}) WHERE id = $1 `;
-            query += `RETURNING *`;
+            const columns = PostgresUtils_1.default.columns(table).filter(column => {
+                const camelKey = PostgresUtils_1.default.stringToCamel(column);
+                return Object.prototype.hasOwnProperty.call(object, camelKey);
+            });
+            if (columns.length === 0)
+                throw new Error(`No valid columns provided to update for table ${tableName}`);
+            const params = columns.map((_, index) => `\$${index + 2}`).join(",");
+            const values = [id, ...columns.map((column) => object[PostgresUtils_1.default.stringToCamel(column)])];
+            const query = `UPDATE ${schema}.${tableName} SET (${columns.join(",")}) = (${params}) WHERE id = $1 RETURNING *`;
             return this.query(query, values).then((result) => result[0]);
         });
     }

package/dist/types/repository/RepositoryDatabasePostgres.d.ts

@@ -36,9 +36,7 @@ export default class RepositoryDatabasePostgres extends RepositoryDatabase {
     addAll<T>(tableName: string, objects: ({
         [key: string]: Primitive;
     } | any)[]): Promise<T[]>;
-    update<T>(tableName: string, id: string, object: {
-        [key: string]: Primitive;
-    } | any): Promise<T>;
+    update<T>(tableName: string, id: string, object: Partial<T>): Promise<T>;
     page<T>(tableName: string, sortKey: string, maxResults?: number, pageNumber?: number, filters?: {
         [key: string]: Primitive | Array<Primitive>;
     }): Promise<Pagination<T>>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ismx-nexo-node-app",
-  "version": "0.4.18",
+  "version": "0.4.19",
   "description": "",
   "scripts": {
     "build": "rm -rf ./dist && npx tsc",

package/src/main/node/repository/RepositoryDatabasePostgres.ts

@@ -120,18 +120,22 @@ export default class RepositoryDatabasePostgres extends RepositoryDatabase
         return this.query(query, values);
     }
 
-    async update<T>(tableName: string, id: string, object: {[key:string]:Primitive}|any): Promise<T>
+    async update<T>(tableName: string, id: string, object: Partial<T>): Promise<T>
     {
         if (!id) throw new Error(`field 'id' is mandatory when updating ${tableName}`);
         if (!this.tables[tableName]) throw new Error(`table ${tableName} does not exist`);
         let table = this.tables[tableName];
         let schema = table[0].schema;
-        let columns = PostgresUtils.columns(table);
+        const columns = PostgresUtils.columns(table).filter(column => {
+            const camelKey = PostgresUtils.stringToCamel(column);
+            return Object.prototype.hasOwnProperty.call(object, camelKey);
+        });
+
+        if (columns.length === 0) throw new Error(`No valid columns provided to update for table ${tableName}`);
 
-        let params = columns.map((column, index) => `\$${index+2}`).join(",");
-        let values = [ id, ...columns.map((column) => object[PostgresUtils.stringToCamel(column)]) ];
-        let query = `UPDATE ${schema}.${tableName} SET (${columns.join(",")}) = (${params}) WHERE id = $1 `
-        query += `RETURNING *`;
+        const params = columns.map((_, index) => `\$${index + 2}`).join(",");
+        const values = [ id, ...columns.map((column) => object[PostgresUtils.stringToCamel(column) as keyof T]) ];
+        const query = `UPDATE ${schema}.${tableName} SET (${columns.join(",")}) = (${params}) WHERE id = $1 RETURNING *`;
 
         return this.query<T>(query, values).then((result) => result[0]);
     }