@saltcorn/data 1.1.3 → 1.1.4-alpha.1

package/dist/models/table.js

@@ -222,13 +222,17 @@ class Table {
         t.update = async (upd_rec) => {
             const { fields, constraints, ...updDB } = upd_rec;
             await db_1.default.update("_sc_tables", updDB, tbl.id);
-            await require("../db/state").getState().refresh_tables();
+            //limited refresh if we do not have a client
+            if (!db_1.default.getRequestContext()?.client)
+                await require("../db/state").getState().refresh_tables(true);
         };
         t.delete = async (upd_rec) => {
             const schema = db_1.default.getTenantSchemaPrefix();
             await db_1.default.deleteWhere("_sc_tag_entries", { table_id: this.id });
             await db_1.default.query(`delete FROM ${schema}_sc_tables WHERE id = $1`, [tbl.id]);
-            await require("../db/state").getState().refresh_tables();
+            //limited refresh if we do not have a client
+            if (!db_1.default.getRequestContext()?.client)
+                await require("../db/state").getState().refresh_tables(true);
         };
         return t;
     }
@@ -609,7 +613,9 @@ class Table {
         if (table.has_sync_info)
             await table.create_sync_info_table();
         // refresh tables cache
-        await require("../db/state").getState().refresh_tables();
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
         return table;
     }
     /**
@@ -653,34 +659,23 @@ class Table {
         const schema = db_1.default.getTenantSchemaPrefix();
         const is_sqlite = db_1.default.isSQLite;
         await this.update({ ownership_field_id: null });
-        const client = is_sqlite ? db_1.default : await db_1.default.getClient();
-        await client.query(`BEGIN`);
-        try {
-            // drop table
-            if (!only_forget)
-                await client.query(`drop table if exists ${schema}"${(0, internal_1.sqlsanitize)(this.name)}"`);
-            // delete tag entries from _sc_tag_entries
-            await db_1.default.deleteWhere("_sc_tag_entries", { table_id: this.id });
-            // delete fields
-            await client.query(`delete FROM ${schema}_sc_fields WHERE table_id = $1`, [this.id]);
-            // delete table description
-            await client.query(`delete FROM ${schema}_sc_tables WHERE id = $1`, [
-                this.id,
-            ]);
-            // delete versioned table
-            if (this.versioned)
-                await client.query(`drop table if exists ${schema}"${(0, internal_1.sqlsanitize)(this.name)}__history"`);
-            await client.query(`COMMIT`);
-        }
-        catch (e) {
-            await client.query(`ROLLBACK`);
-            if (!is_sqlite)
-                client.release(true);
-            throw e;
-        }
-        if (!is_sqlite)
-            client.release(true);
-        await require("../db/state").getState().refresh_tables();
+        // drop table
+        if (!only_forget)
+            await db_1.default.query(`drop table if exists ${schema}"${(0, internal_1.sqlsanitize)(this.name)}"`);
+        // delete tag entries from _sc_tag_entries
+        await db_1.default.deleteWhere("_sc_tag_entries", { table_id: this.id });
+        // delete fields
+        await db_1.default.query(`delete FROM ${schema}_sc_fields WHERE table_id = $1`, [
+            this.id,
+        ]);
+        // delete table description
+        await db_1.default.query(`delete FROM ${schema}_sc_tables WHERE id = $1`, [this.id]);
+        // delete versioned table
+        if (this.versioned)
+            await db_1.default.query(`drop table if exists ${schema}"${(0, internal_1.sqlsanitize)(this.name)}__history"`);
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
     }
     /***
      * Get Table SQL Name
@@ -1958,27 +1953,18 @@ class Table {
         if (db_1.default.isSQLite)
             throw new InvalidAdminAction("Cannot rename table on SQLite");
         const schemaPrefix = db_1.default.getTenantSchemaPrefix();
-        const client = await db_1.default.getClient();
-        await client.query(`BEGIN`);
-        try {
-            //rename table
-            await db_1.default.query(`alter table ${schemaPrefix}"${(0, internal_1.sqlsanitize)(this.name)}" rename to "${(0, internal_1.sqlsanitize)(new_name)}";`);
-            //change refs
-            await db_1.default.query(`update ${schemaPrefix}_sc_fields set reftable_name=$1 where reftable_name=$2`, [new_name, this.name]);
-            //rename history
-            if (this.versioned)
-                await db_1.default.query(`alter table ${schemaPrefix}"${(0, internal_1.sqlsanitize)(this.name)}__history" rename to "${(0, internal_1.sqlsanitize)(new_name)}__history";`);
-            //1. change record
-            await this.update({ name: new_name });
-            await client.query(`COMMIT`);
-        }
-        catch (e) {
-            await client.query(`ROLLBACK`);
-            client.release(true);
-            throw e;
-        }
-        client.release(true);
-        await require("../db/state").getState().refresh_tables();
+        //rename table
+        await db_1.default.query(`alter table ${schemaPrefix}"${(0, internal_1.sqlsanitize)(this.name)}" rename to "${(0, internal_1.sqlsanitize)(new_name)}";`);
+        //change refs
+        await db_1.default.query(`update ${schemaPrefix}_sc_fields set reftable_name=$1 where reftable_name=$2`, [new_name, this.name]);
+        //rename history
+        if (this.versioned)
+            await db_1.default.query(`alter table ${schemaPrefix}"${(0, internal_1.sqlsanitize)(this.name)}__history" rename to "${(0, internal_1.sqlsanitize)(new_name)}__history";`);
+        //1. change record
+        await this.update({ name: new_name });
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
     }
     /**
      * Update Table description in _sc_table
@@ -1995,8 +1981,10 @@ class Table {
         }
         const { external, fields, constraints, ...upd_rec } = new_table_rec;
         await db_1.default.update("_sc_tables", upd_rec, this.id);
-        await require("../db/state").getState().refresh_tables();
-        const new_table = Table.findOne({ id: this.id });
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
+        const new_table = new Table({ ...this, ...upd_rec });
         if (!new_table) {
             throw new Error(`Unable to find table with id: ${this.id}`);
         }
@@ -2016,6 +2004,9 @@ class Table {
             Object.assign(this, new_table_rec);
         }
     }
+    static async state_refresh() {
+        await require("../db/state").getState().refresh_tables();
+    }
     /**
      * Get table history data
      * @param id
@@ -2116,7 +2107,9 @@ class Table {
             return { error: parse_res.error };
         }
         parse_res.table = table;
-        await require("../db/state").getState().refresh_tables();
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
         return parse_res;
     }
     /**
@@ -3198,7 +3191,9 @@ where table_schema = '${db_1.default.getTenantSchema() || "public"}'
             delete attrs.NonSerial;
             await pk.update({ attributes: attrs });
         }
-        await require("../db/state").getState().refresh_tables();
+        //limited refresh if we do not have a client
+        if (!db_1.default.getRequestContext()?.client)
+            await require("../db/state").getState().refresh_tables(true);
     }
     async move_include_fts_to_search_context() {
         const include_fts_fields = this.fields.filter((f) => f.attributes?.include_fts);