sedentary-pg 0.0.46 → 0.0.49

package/dist/cjs/pgdb.js

@@ -102,10 +102,10 @@ class PGDB extends sedentary_1.DB {
         return (0, pg_format_1.default)("%L", value);
     }
     fill(attributes, row, entry) {
-        const loaded = {};
+        const value = {};
         for (const attribute in attributes)
-            entry[attribute] = loaded[attribute] = row[attributes[attribute]];
-        Object.defineProperty(entry, "loaded", { configurable: true, value: loaded });
+            entry[attribute] = value[attribute] = row[attributes[attribute]];
+        Object.defineProperty(entry, sedentary_1.loaded, { configurable: true, value });
     }
     load(tableName, attributes, pk, model, table) {
         const pkFldName = pk.fieldName;
@@ -158,7 +158,7 @@ class PGDB extends sedentary_1.DB {
         const pkAttrName = pk.attributeName;
         const pkFldName = pk.fieldName;
         return async function () {
-            const client = this.tx ? this.tx._client : await self.pool.connect();
+            const client = this[sedentary_1.transaction] ? this[sedentary_1.transaction]._client : await self.pool.connect();
             let removed;
             try {
                 const query = `DELETE FROM ${tableName} WHERE ${pkFldName} = ${self.escape(this[pkAttrName])}`;
@@ -166,7 +166,7 @@ class PGDB extends sedentary_1.DB {
                 removed = (await client.query(query)).rowCount;
             }
             finally {
-                if (!this.tx)
+                if (!this[sedentary_1.transaction])
                     client.release();
             }
             return removed;
@@ -178,7 +178,7 @@ class PGDB extends sedentary_1.DB {
         const pkAttrName = pk.attributeName;
         const pkFldName = pk.fieldName;
         return async function () {
-            const client = this.tx ? this.tx._client : await self.pool.connect();
+            const client = this[sedentary_1.transaction] ? this[sedentary_1.transaction]._client : await self.pool.connect();
             let changed = false;
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             let result = null;
@@ -189,12 +189,12 @@ class PGDB extends sedentary_1.DB {
                 self.fill(attributes, result.rows[0], this);
             };
             try {
-                const { loaded } = this;
-                if (loaded) {
+                const loadedRecord = this[sedentary_1.loaded];
+                if (loadedRecord) {
                     const actions = [];
                     for (const attribute in attributes) {
                         const value = this[attribute];
-                        if (value !== loaded[attribute])
+                        if ((0, sedentary_1.differ)(value, loadedRecord[attribute]))
                             actions.push(`${attributes[attribute]} = ${self.escape(value)}`);
                     }
                     if (actions.length)
@@ -214,7 +214,7 @@ class PGDB extends sedentary_1.DB {
                 }
             }
             finally {
-                if (!this.tx)
+                if (!this[sedentary_1.transaction])
                     client.release();
             }
             return changed && result.rowCount;
@@ -252,9 +252,11 @@ class PGDB extends sedentary_1.DB {
     }
     async dropFields(table) {
         const res = await this._client.query("SELECT attname FROM pg_attribute WHERE attrelid = $1 AND attnum > 0 AND attisdropped = false AND attinhcount = 0", [table.oid]);
-        for (const i in res.rows)
-            if (!table.findField(res.rows[i].attname))
+        for (const i in res.rows) {
+            const field = table.findField(res.rows[i].attname);
+            if (!field || !field.base)
                 await this.dropField(table.tableName, res.rows[i].attname);
+        }
     }
     async dropIndexes(table, constraintIndexes) {
         const { indexes, oid } = table;
@@ -337,6 +339,8 @@ class PGDB extends sedentary_1.DB {
                 return ["BIGINT", "BIGINT"];
             case "JSON":
                 return ["JSON", "JSON"];
+            case "NONE":
+                return ["NONE", "NONE"];
             case "NUMBER":
                 return ["NUMERIC", "NUMERIC"];
             case "VARCHAR":
@@ -388,8 +392,10 @@ class PGDB extends sedentary_1.DB {
                 await setNotNull(isNotNull);
             };
             if (!res.rowCount) {
-                await addField();
-                await setDefault(false);
+                if (type !== "NONE") {
+                    await addField();
+                    await setDefault(false);
+                }
             }
             else {
                 const { adsrc, attnotnull, atttypmod, typname } = res.rows[0];

package/dist/es/pgdb.js

@@ -1,6 +1,6 @@
 import { DatabaseError, Pool, types as PGtypes } from "pg";
 import format from "pg-format";
-import { DB, Transaction } from "sedentary";
+import { DB, differ, loaded, Transaction, transaction } from "sedentary";
 import { adsrc } from "./adsrc";
 const needDrop = [
     ["DATETIME", "int2"],
@@ -97,10 +97,10 @@ export class PGDB extends DB {
         return format("%L", value);
     }
     fill(attributes, row, entry) {
-        const loaded = {};
+        const value = {};
         for (const attribute in attributes)
-            entry[attribute] = loaded[attribute] = row[attributes[attribute]];
-        Object.defineProperty(entry, "loaded", { configurable: true, value: loaded });
+            entry[attribute] = value[attribute] = row[attributes[attribute]];
+        Object.defineProperty(entry, loaded, { configurable: true, value });
     }
     load(tableName, attributes, pk, model, table) {
         const pkFldName = pk.fieldName;
@@ -153,7 +153,7 @@ export class PGDB extends DB {
         const pkAttrName = pk.attributeName;
         const pkFldName = pk.fieldName;
         return async function () {
-            const client = this.tx ? this.tx._client : await self.pool.connect();
+            const client = this[transaction] ? this[transaction]._client : await self.pool.connect();
             let removed;
             try {
                 const query = `DELETE FROM ${tableName} WHERE ${pkFldName} = ${self.escape(this[pkAttrName])}`;
@@ -161,7 +161,7 @@ export class PGDB extends DB {
                 removed = (await client.query(query)).rowCount;
             }
             finally {
-                if (!this.tx)
+                if (!this[transaction])
                     client.release();
             }
             return removed;
@@ -173,7 +173,7 @@ export class PGDB extends DB {
         const pkAttrName = pk.attributeName;
         const pkFldName = pk.fieldName;
         return async function () {
-            const client = this.tx ? this.tx._client : await self.pool.connect();
+            const client = this[transaction] ? this[transaction]._client : await self.pool.connect();
             let changed = false;
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             let result = null;
@@ -184,12 +184,12 @@ export class PGDB extends DB {
                 self.fill(attributes, result.rows[0], this);
             };
             try {
-                const { loaded } = this;
-                if (loaded) {
+                const loadedRecord = this[loaded];
+                if (loadedRecord) {
                     const actions = [];
                     for (const attribute in attributes) {
                         const value = this[attribute];
-                        if (value !== loaded[attribute])
+                        if (differ(value, loadedRecord[attribute]))
                             actions.push(`${attributes[attribute]} = ${self.escape(value)}`);
                     }
                     if (actions.length)
@@ -209,7 +209,7 @@ export class PGDB extends DB {
                 }
             }
             finally {
-                if (!this.tx)
+                if (!this[transaction])
                     client.release();
             }
             return changed && result.rowCount;
@@ -247,9 +247,11 @@ export class PGDB extends DB {
     }
     async dropFields(table) {
         const res = await this._client.query("SELECT attname FROM pg_attribute WHERE attrelid = $1 AND attnum > 0 AND attisdropped = false AND attinhcount = 0", [table.oid]);
-        for (const i in res.rows)
-            if (!table.findField(res.rows[i].attname))
+        for (const i in res.rows) {
+            const field = table.findField(res.rows[i].attname);
+            if (!field || !field.base)
                 await this.dropField(table.tableName, res.rows[i].attname);
+        }
     }
     async dropIndexes(table, constraintIndexes) {
         const { indexes, oid } = table;
@@ -332,6 +334,8 @@ export class PGDB extends DB {
                 return ["BIGINT", "BIGINT"];
             case "JSON":
                 return ["JSON", "JSON"];
+            case "NONE":
+                return ["NONE", "NONE"];
             case "NUMBER":
                 return ["NUMERIC", "NUMERIC"];
             case "VARCHAR":
@@ -383,8 +387,10 @@ export class PGDB extends DB {
                 await setNotNull(isNotNull);
             };
             if (!res.rowCount) {
-                await addField();
-                await setDefault(false);
+                if (type !== "NONE") {
+                    await addField();
+                    await setDefault(false);
+                }
             }
             else {
                 const { adsrc, attnotnull, atttypmod, typname } = res.rows[0];

package/dist/types/pgdb.d.ts

@@ -1,5 +1,5 @@
 import { PoolClient, PoolConfig } from "pg";
-import { Attribute, DB, EntryBase, Table, Transaction } from "sedentary";
+import { Attribute, DB, EntryBase, loaded, Table, Transaction, transaction } from "sedentary";
 export declare class PGDB extends DB<TransactionPG> {
     private _client;
     private indexes;
@@ -18,11 +18,11 @@ export declare class PGDB extends DB<TransactionPG> {
     fill(attributes: Record<string, string>, row: Record<string, unknown>, entry: Record<string, unknown>): void;
     load(tableName: string, attributes: Record<string, string>, pk: Attribute<unknown, unknown>, model: new (from: "load") => EntryBase, table: Table): (where: string, order?: string | string[], limit?: number, tx?: Transaction) => Promise<EntryBase[]>;
     remove(tableName: string, pk: Attribute<unknown, unknown>): (this: Record<string, unknown> & {
-        tx?: TransactionPG;
+        [transaction]?: TransactionPG;
     }) => Promise<number>;
     save(tableName: string, attributes: Record<string, string>, pk: Attribute<unknown, unknown>): (this: Record<string, unknown> & {
-        loaded?: Record<string, unknown>;
-        tx?: TransactionPG;
+        [loaded]?: Record<string, unknown>;
+        [transaction]?: TransactionPG;
     }) => Promise<number | false>;
     dropConstraints(table: Table): Promise<number[]>;
     dropField(tableName: string, fieldName: string): Promise<void>;

package/package.json

@@ -9,7 +9,7 @@
     "@types/pg": "8.6.5",
     "pg": "8.8.0",
     "pg-format": "1.0.4",
-    "sedentary": "0.0.46"
+    "sedentary": "0.0.49"
   },
   "description": "The ORM which never needs to migrate - PostgreSQL",
   "engines": {
@@ -47,5 +47,5 @@
     "test": "jest --no-cache --runInBand"
   },
   "types": "./dist/types/index.d.ts",
-  "version": "0.0.46"
+  "version": "0.0.49"
 }