@dwtechs/antity-pgsql 0.6.0 → 0.7.0

package/README.md

@@ -2,7 +2,7 @@
 [![License: MIT](https://img.shields.io/npm/l/@dwtechs/antity-pgsql.svg?color=brightgreen)](https://opensource.org/licenses/MIT)
 [![npm version](https://badge.fury.io/js/%40dwtechs%2Fantity-pgsql.svg)](https://www.npmjs.com/package/@dwtechs/antity-pgsql)
 [![last version release date](https://img.shields.io/github/release-date/DWTechs/Antity-pgsql.js)](https://www.npmjs.com/package/@dwtechs/antity-pgsql)
-![Jest:coverage](https://img.shields.io/badge/Jest:coverage-100%25-brightgreen.svg)
+![Jest:coverage](https://img.shields.io/badge/Jest:coverage-54%25-brightgreen.svg)
 
 - [Synopsis](#synopsis)
 - [Support](#support)
@@ -123,7 +123,9 @@ const entity = new Entity("consumers", [
 router.get("/", ..., entity.get);
 router.post("/", entity.normalizeArray, entity.validateArray, ..., entity.add);
 router.put("/", entity.normalizeArray, entity.validateArray, ..., entity.update);
-router.put("/", ..., entity.archive);
+router.put("/archive", ..., entity.archive);
+router.delete("/", ..., entity.delete);
+router.delete("/archived", ..., entity.deleteArchive);
 
 ```
 
@@ -181,29 +183,34 @@ class SQLEntity {
         query: string;
         args: (Filter["value"])[];
       };
-      update: (
-        rows: Record<string, unknown>[],
-        consumerId?: number | string,
-        consumerName?: string) => {
-          query: string;
+    update: (
+      rows: Record<string, unknown>[],
+      consumerId?: number | string,
+      consumerName?: string) => {
+        query: string;
+        args: unknown[];
+    };
+    insert: (
+      rows: Record<string, unknown>[],
+      consumerId?: number | string,
+      consumerName?: string,
+      rtn?: string) => {
+        query: string;
           args: unknown[];
       };
-      insert: (
-        rows: Record<string, unknown>[],
-        consumerId?: number | string,
-        consumerName?: string,
-        rtn?: string) => {
-          query: string;
-            args: unknown[];
-        };
-        delete: () => string;
-        return: (prop: string) => string;
+    delete: (ids: number[]) => {
+      query: string;
+      args: number[];
+    };
+    deleteArchive: () => string;
+    return: (prop: string) => string;
   };
   get: (req: Request, res: Response, next: NextFunction) => void;
   add: (req: Request, res: Response, next: NextFunction) => Promise<void>;
   update: (req: Request, res: Response, next: NextFunction) => Promise<void>;
   archive: (req: Request, res: Response, next: NextFunction) => Promise<void>;
-  delete: (req: Request, res: Response, next: NextFunction) => void;
+  delete: (req: Request, res: Response, next: NextFunction) => Promise<void>;
+  deleteArchive: (req: Request, res: Response, next: NextFunction) => void;
 
 }
 
@@ -223,9 +230,12 @@ function execute(
 
 
 ```
-get(), add(), update(), archive() and delete() methods are made to be used as Express.js middlewares.
+get(), add(), update(), archive(), delete() and deleteArchive() methods are made to be used as Express.js middlewares.
 Each method will look for data to work on in the **req.body.rows** parameter.
 
+- **delete()**: Deletes rows by their IDs. Expects `req.body.rows` to be an array of objects with `id` property: `[{id: 1}, {id: 2}]`
+- **deleteArchive()**: Deletes archived rows that were archived before a specific date. Expects `req.body.date` to be a Date object.
+
 
 ## Match modes
 

package/dist/antity-pgsql.d.ts

@@ -96,14 +96,19 @@ declare class SQLEntity extends Entity {
         query: string;
         args: unknown[];
     };
-    delete: () => string;
+    delete: (ids: number[]) => {
+      query: string;
+      args: number[];
+    };
+    deleteArchive: () => string;
     return: (prop: string) => string;
   };
   get: (req: Request, res: Response, next: NextFunction) => void;
   add: (req: Request, res: Response, next: NextFunction) => Promise<void>;
   update: (req: Request, res: Response, next: NextFunction) => Promise<void>;
   archive: (req: Request, res: Response, next: NextFunction) => Promise<void>;
-  delete: (req: Request, res: Response, next: NextFunction) => void;
+  delete: (req: Request, res: Response, next: NextFunction) => Promise<void>;
+  deleteArchive: (req: Request, res: Response, next: NextFunction) => void;
 }
 
 declare function filter(

package/dist/antity-pgsql.js

@@ -58,7 +58,7 @@ var perf = {
     end,
 };
 
-function execute$1(query, args, clt) {
+function execute(query, args, clt) {
     const time = perf.start(query, args);
     const client = clt || pool;
     return client
@@ -233,7 +233,7 @@ class Select {
         };
     }
     execute(query, args, client) {
-        return execute$1(query, args, client)
+        return execute(query, args, client)
             .then((r) => {
             if (!r.rowCount)
                 throw { status: 404, msg: "Resource not found" };
@@ -293,10 +293,10 @@ class Insert {
         return { query, args };
     }
     rtn(prop) {
-        return `RETURNING "${prop}"`;
+        return `RETURNING ${quoteIfUppercase(prop)}`;
     }
     execute(query, args, client) {
-        return execute$1(query, args, client);
+        return execute(query, args, client);
     }
 }
 
@@ -314,16 +314,15 @@ class Update {
         this._props = [];
     }
     addProp(prop) {
-        this._props.push(quoteIfUppercase(prop));
+        this._props.push(prop);
     }
     query(table, rows, consumerId, consumerName) {
-        if (consumerId !== undefined && consumerName !== undefined) {
+        if (consumerId !== undefined && consumerName !== undefined)
             rows = this.addConsumer(rows, consumerId, consumerName);
-        }
         const propsToUse = [...this._props];
-        if (consumerId !== undefined && consumerName !== undefined) {
+        if (consumerId !== undefined && consumerName !== undefined)
             propsToUse.push("consumerId", "consumerName");
-        }
+        log.debug(`${LOGS_PREFIX}Update query input rows: ${JSON.stringify(rows, null, 2)}`);
         const l = rows.length;
         const args = rows.map(row => row.id);
         let query = `UPDATE "${quoteIfUppercase(table)}" SET `;
@@ -331,7 +330,7 @@ class Update {
         for (const p of propsToUse) {
             if (rows[0][p] === undefined)
                 continue;
-            query += `${p} = CASE `;
+            query += `${quoteIfUppercase(p)} = CASE `;
             for (let j = 0; j < l; j++) {
                 const row = rows[j];
                 query += `WHEN id = $${j + 1} THEN $${i++} `;
@@ -344,7 +343,7 @@ class Update {
     }
     execute(query, args, client) {
         return __awaiter$2(this, void 0, void 0, function* () {
-            return execute$1(query, args, client);
+            return execute(query, args, client);
         });
     }
     addConsumer(rows, consumerId, consumerName) {
@@ -362,14 +361,20 @@ var __awaiter$1 = (undefined && undefined.__awaiter) || function (thisArg, _argu
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-function query(table) {
+function queryById(table, ids) {
+    return {
+        query: `DELETE FROM ${quoteIfUppercase(table)} WHERE id = ANY($1)`,
+        args: [ids]
+    };
+}
+function queryArchived(table) {
     return `DELETE FROM ${quoteIfUppercase(table)} WHERE "archivedAt" < $1`;
 }
-function execute(date, query, client) {
+function executeArchived(date, query, client) {
     return __awaiter$1(this, void 0, void 0, function* () {
         let db;
         try {
-            db = yield execute$1(query, [date], client);
+            db = yield execute(query, [date], client);
         }
         catch (err) {
             throw err;
@@ -562,8 +567,11 @@ class SQLEntity extends Entity {
             insert: (rows, consumerId, consumerName, rtn = "") => {
                 return this.ins.query(this.table, rows, consumerId, consumerName, rtn);
             },
-            delete: () => {
-                return query(this.table);
+            delete: (ids) => {
+                return queryById(this.table, ids);
+            },
+            deleteArchive: () => {
+                return queryArchived(this.table);
             },
             return: (prop) => {
                 return this.ins.rtn(prop);
@@ -605,7 +613,7 @@ class SQLEntity extends Entity {
                 const { query, args } = this.ins.query(this._table, c, cId, cName, rtn);
                 let db;
                 try {
-                    db = yield execute$1(query, args, dbClient);
+                    db = yield execute(query, args, dbClient);
                 }
                 catch (err) {
                     return next(err);
@@ -629,7 +637,7 @@ class SQLEntity extends Entity {
             for (const c of chunks) {
                 const { query, args } = this.upd.query(this._table, c, cId, cName);
                 try {
-                    yield execute$1(query, args, dbClient);
+                    yield execute(query, args, dbClient);
                 }
                 catch (err) {
                     return next(err);
@@ -649,7 +657,7 @@ class SQLEntity extends Entity {
             for (const c of chunks) {
                 const { query, args } = this.upd.query(this._table, c, cId, cName);
                 try {
-                    yield execute$1(query, args, dbClient);
+                    yield execute(query, args, dbClient);
                 }
                 catch (err) {
                     return next(err);
@@ -657,12 +665,26 @@ class SQLEntity extends Entity {
             }
             next();
         });
-        this.delete = (req, res, next) => {
+        this.delete = (req, res, next) => __awaiter(this, void 0, void 0, function* () {
+            const rows = req.body.rows;
+            const dbClient = res.locals.dbClient || null;
+            const ids = rows.map((row) => row.id);
+            log.debug(`${LOGS_PREFIX}delete ${rows.length} rows : (${ids.join(", ")})`);
+            const { query, args } = queryById(this._table, ids);
+            try {
+                yield execute(query, args, dbClient);
+            }
+            catch (err) {
+                return next(err);
+            }
+            next();
+        });
+        this.deleteArchive = (req, res, next) => {
             const date = req.body.date;
             const dbClient = res.locals.dbClient || null;
-            log.debug(`${LOGS_PREFIX}delete archived`);
-            const q = query(this._table);
-            execute(date, q, dbClient)
+            log.debug(`${LOGS_PREFIX}deleteArchive(date=${date})`);
+            const q = queryArchived(this._table);
+            executeArchived(date, q, dbClient)
                 .then(() => next())
                 .catch((err) => next(err));
         };
@@ -698,4 +720,4 @@ class SQLEntity extends Entity {
     }
 }
 
-export { SQLEntity, execute$1 as execute, filter };
+export { SQLEntity, execute, filter };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dwtechs/antity-pgsql",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Open source library to add PostgreSQL support to @dwtechs/Antity entities.",
   "keywords": [
     "entities"