@saltcorn/data 1.1.3 → 1.1.4-alpha.1

package/dist/base-plugin/index.d.ts

@@ -1139,9 +1139,7 @@ export const viewtemplates: ({
         authorizeGetQuery(query: any, table_id: any): Promise<string | number | boolean | null | undefined>;
         getRowQuery(table_id: any, view_select: any, row_id: any, order_field: any): Promise<import("@saltcorn/db-common/internal").Row[]>;
         getRowByIdQuery(id: any): Promise<import("@saltcorn/db-common/internal").Row | null>;
-        actionQuery(): Promise<{
-            json: any;
-        }>;
+        actionQuery(): Promise<any>;
         optionsQuery(reftable_name: any, type: any, attributes: any, where: any): Promise<import("@saltcorn/db-common/internal").Row[]>;
         updateMatchingQuery(where: any, updateVals: any, repeatFields: any, childRows: any): Promise<any[] | {
             rowError: any;
@@ -1262,9 +1260,7 @@ export const viewtemplates: ({
             orderBy: any;
             orderDesc: any;
         }): Promise<import("@saltcorn/db-common/internal").Row[]>;
-        actionQuery(): Promise<{
-            json: any;
-        }>;
+        actionQuery(): Promise<any>;
     };
     configCheck: (view: any) => Promise<{
         errors: string[];
@@ -1375,9 +1371,7 @@ export const viewtemplates: ({
         exttable_name: any;
     }) => {
         optionsQuery(reftable_name: any, type: any, attributes: any, whereWithExisting: any, user: any): Promise<any>;
-        actionQuery(state: any, rndid: any): Promise<{
-            json: any;
-        }>;
+        actionQuery(state: any, rndid: any): Promise<any>;
         distinctValuesQuery(state: any): Promise<{
             distinct_values: {};
             role: any;

package/dist/base-plugin/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../base-plugin/index.js"],"names":[],"mappings":"AAwCA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;KAAsD;AACtD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;KASE;;;;0CAGW,MAAM"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../base-plugin/index.js"],"names":[],"mappings":"AAwCA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;KAAsD;AACtD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;KASE;;;;0CAGW,MAAM"}

package/dist/base-plugin/viewtemplates/edit.d.ts

@@ -209,9 +209,7 @@ export declare function queries({ table_id, name, configuration: { columns, defa
     authorizeGetQuery(query: any, table_id: any): Promise<string | number | boolean | null | undefined>;
     getRowQuery(table_id: any, view_select: any, row_id: any, order_field: any): Promise<import("@saltcorn/db-common/internal").Row[]>;
     getRowByIdQuery(id: any): Promise<import("@saltcorn/db-common/internal").Row | null>;
-    actionQuery(): Promise<{
-        json: any;
-    }>;
+    actionQuery(): Promise<any>;
     optionsQuery(reftable_name: any, type: any, attributes: any, where: any): Promise<import("@saltcorn/db-common/internal").Row[]>;
     updateMatchingQuery(where: any, updateVals: any, repeatFields: any, childRows: any): Promise<any[] | {
         rowError: any;

package/dist/base-plugin/viewtemplates/edit.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"edit.d.ts","sourceRoot":"","sources":["../../../base-plugin/viewtemplates/edit.js"],"names":[],"mappings":"AAqGA;;;GAGG;AAEH,4CAJW,MAAM,GACJ,QAAQ,CAgXjB;AA+CJ;;;;;;;;;;;GAWG;AACH,8BAXW,MAAM,YACN,MAAM,mBAIN,MAAM;IAEU,GAAG,EAAnB,MAAM;IACU,GAAG,EAAnB,MAAM;;;IACJ,QAAQ,IAAI,CAAC,CAYzB;AAED;;;;;;;;;GASG;AACH,kCATW,MAAM,YACN,MAAM;IAEE,OAAO;IACP,MAAM;wBAEd,MAAM;;;;IACJ,QAAQ,IAAI,EAAE,CAAC,CA0C3B;AA6jBD;;;;;;;;;;;;;;;;GAgBG;AACH,kCAhBW,MAAM,YACN,MAAM;IAEY,OAAO,EAAzB,MAAM,EAAE;IACQ,MAAM;IACN,KAAK,EAArB,MAAM;IACW,cAAc,EAA/B,OAAO;IACW,oBAAoB,EAAtC,MAAM,EAAE;UACR,MAAM;IAGU,GAAG,EAAnB,MAAM;IACU,GAAG,EAAnB,MAAM;IACU,QAAQ,EAAxB,MAAM;;;;;;;;;iBACJ,QAAQ,IAAI,CAAC,CAuTzB;AA0FD,mKA6BC;AAGD,sHA6BC;AAjpCD;;;;;;GAMG;AACH;IAHmB,OAAO;IACb,QAAQ,MAAM,EAAE,CAAC,CAQ7B;AA6BD,uBAAuB;AACvB,sCAAuD;AA2hCvD;;;;;;GAMG;AACH;IALwB,IAAI,EAAjB,MAAM;IACO,QAAQ,EAArB,MAAM;IACO,GAAG,EAAhB,MAAM;;;IACJ,QAAQ,OAAO,CAAC,CAO5B;;;;AAiED;;;;;;;;;;;GAWG;AACH,sCAXW,MAAM;IAGS,OAAO,EAAtB,MAAM,EAAE;IACA,MAAM;;IAGE,GAAG,EAAnB,MAAM;IACK,GAAG;;;IACZ,QAAQ,MAAM,CAAC,CAe3B;AAED;;;;;;;;;;;;;;;;;;;;;;kBA0GC;;yBAiWY,MAAM;gCAEN,MAAM;AAiBF;;;;;qBAEd;AACD;;;;GAIG;AACH;;aAEC;AACQ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EA+bP;;;;;AAEF,yGAeC;AACY;;;GA2DZ;AACiB,6HAEjB"}
+{"version":3,"file":"edit.d.ts","sourceRoot":"","sources":["../../../base-plugin/viewtemplates/edit.js"],"names":[],"mappings":"AAqGA;;;GAGG;AAEH,4CAJW,MAAM,GACJ,QAAQ,CAgXjB;AA+CJ;;;;;;;;;;;GAWG;AACH,8BAXW,MAAM,YACN,MAAM,mBAIN,MAAM;IAEU,GAAG,EAAnB,MAAM;IACU,GAAG,EAAnB,MAAM;;;IACJ,QAAQ,IAAI,CAAC,CAYzB;AAED;;;;;;;;;GASG;AACH,kCATW,MAAM,YACN,MAAM;IAEE,OAAO;IACP,MAAM;wBAEd,MAAM;;;;IACJ,QAAQ,IAAI,EAAE,CAAC,CA0C3B;AA6jBD;;;;;;;;;;;;;;;;GAgBG;AACH,kCAhBW,MAAM,YACN,MAAM;IAEY,OAAO,EAAzB,MAAM,EAAE;IACQ,MAAM;IACN,KAAK,EAArB,MAAM;IACW,cAAc,EAA/B,OAAO;IACW,oBAAoB,EAAtC,MAAM,EAAE;UACR,MAAM;IAGU,GAAG,EAAnB,MAAM;IACU,GAAG,EAAnB,MAAM;IACU,QAAQ,EAAxB,MAAM;;;;;;;;;iBACJ,QAAQ,IAAI,CAAC,CAgUzB;AA0FD,mKA6BC;AAGD,sHA6BC;AA1pCD;;;;;;GAMG;AACH;IAHmB,OAAO;IACb,QAAQ,MAAM,EAAE,CAAC,CAQ7B;AA6BD,uBAAuB;AACvB,sCAAuD;AAoiCvD;;;;;;GAMG;AACH;IALwB,IAAI,EAAjB,MAAM;IACO,QAAQ,EAArB,MAAM;IACO,GAAG,EAAhB,MAAM;;;IACJ,QAAQ,OAAO,CAAC,CAO5B;;;;AAiED;;;;;;;;;;;GAWG;AACH,sCAXW,MAAM;IAGS,OAAO,EAAtB,MAAM,EAAE;IACA,MAAM;;IAGE,GAAG,EAAnB,MAAM;IACK,GAAG;;;IACZ,QAAQ,MAAM,CAAC,CAe3B;AAED;;;;;;;;;;;;;;;;;;;;;;kBA0GC;;yBAiWY,MAAM;gCAEN,MAAM;AAiBF;;;;;qBAEd;AACD;;;;GAIG;AACH;;aAEC;AACQ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAicP;;;;;AAEF,yGAeC;AACY;;;GA2DZ;AACiB,6HAEjB"}

package/dist/base-plugin/viewtemplates/edit.js

@@ -952,126 +952,133 @@ const runPost = async (table_id, viewname, { columns, layout, fixed, view_when_d
         let ins_upd_error;
         if (!cancel) {
             getState().log(6, `Edit POST ready to insert/update into ${table.name} Row=${JSON.stringify(row)} ID=${id} Ajax=${!!req.xhr}`);
-            if (typeof id === "undefined") {
-                const ins_res = await tryInsertQuery(row);
-                if (ins_res.success) {
-                    id = ins_res.success;
-                    row[pk.name] = id;
-                    trigger_return = ins_res.trigger_return;
-                }
-                else {
-                    ins_upd_error = ins_res.error;
-                }
-            }
-            else {
-                if (table.getField(table.pk_name).attributes.NonSerial) {
-                    const upd_res = await tryInsertOrUpdateImpl(row, id, table, req.user || { role_id: 100 });
-                    if (upd_res.error) {
-                        ins_upd_error = upd_res.error;
+            const doReturn = await db.withTransaction(async (rollback) => {
+                if (typeof id === "undefined") {
+                    const ins_res = await tryInsertQuery(row);
+                    if (ins_res.success) {
+                        id = ins_res.success;
+                        row[pk.name] = id;
+                        trigger_return = ins_res.trigger_return;
                     }
-                    trigger_return = upd_res.trigger_return;
-                }
-                else {
-                    const upd_res = await tryUpdateQuery(row, id);
-                    if (upd_res.error) {
-                        ins_upd_error = upd_res.error;
+                    else {
+                        ins_upd_error = ins_res.error;
                     }
-                    trigger_return = upd_res.trigger_return;
-                }
-            }
-            if (ins_upd_error) {
-                getState().log(6, `Insert or update failure ${JSON.stringify(ins_upd_error)}`);
-                res.status(422);
-                if (req.xhr) {
-                    res.json({ error: ins_upd_error });
                 }
                 else {
-                    await form.fill_fkey_options(false, optionsQuery, req.user);
-                    req.flash("error", text_attr(ins_upd_error));
-                    res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
-                }
-                return;
-            }
-            //Edit-in-edit
-            for (const field of form.fields.filter((f) => f.isRepeat)) {
-                const view_select = parse_view_select(field.metadata.view, field.metadata.relation_path);
-                const order_field = field.metadata.order_field;
-                const childView = View.findOne({ name: view_select.viewname });
-                if (!childView)
-                    throw new InvalidConfiguration(`Cannot find embedded view: ${view_select.viewname}`);
-                if (field.metadata.relation_path &&
-                    view_select.type === "RelationPath") {
-                    const targetTbl = Table.findOne({ id: childView.table_id });
-                    const relation = new Relation(field.metadata.relation_path, targetTbl.name, displayType(await childView.get_state_fields()));
-                    if (relation.type === RelationType.CHILD_LIST)
-                        updateViewSelect(view_select);
-                }
-                const childTable = Table.findOne({ id: field.metadata?.table_id });
-                const submitted_row_ids = new Set((form.values[field.name] || []).map((srow) => `${srow[childTable.pk_name]}`));
-                const childFields = new Set(childTable.fields.map((f) => f.name));
-                let repeatIx = 0;
-                for (const [childRow, row_ix] of form.values[field.name].map((r, ix) => [r, ix])) {
-                    // set fixed here
-                    childRow[field.metadata?.relation] = id;
-                    for (const [k, v] of Object.entries(childView?.configuration?.fixed || {})) {
-                        if (typeof childRow[k] === "undefined" &&
-                            !k.startsWith("_block_") &&
-                            childFields.has(k) &&
-                            (v || v === 0) //no nulls or empty string, but allow 0
-                        )
-                            childRow[k] = v;
-                    }
-                    if (order_field && !childRow[order_field])
-                        childRow[order_field] = row_ix;
-                    for (const file_field of field.fields.filter((f) => f.type === "File")) {
-                        const key = `${file_field.name}_${repeatIx}`;
-                        if (req.files?.[key]) {
-                            const file = await File.from_req_files(req.files[key], req.user ? req.user.id : null, (file_field.attributes &&
-                                +file_field.attributes.min_role_read) ||
-                                1, file_field?.attributes?.folder);
-                            childRow[file_field.name] = file.path_to_serve;
+                    if (table.getField(table.pk_name).attributes.NonSerial) {
+                        const upd_res = await tryInsertOrUpdateImpl(row, id, table, req.user || { role_id: 100 });
+                        if (upd_res.error) {
+                            ins_upd_error = upd_res.error;
                         }
+                        trigger_return = upd_res.trigger_return;
                     }
-                    getState().log(6, `Edit POST ready to insert/update Child row into ${childTable.name} Row=${JSON.stringify(childRow)} ID=${childRow[childTable.pk_name]} Ajax=${!!req.xhr}`);
-                    if (childRow[childTable.pk_name]) {
-                        const upd_res = await childTable.tryUpdateRow(childRow, childRow[childTable.pk_name], req.user || { role_id: 100 });
+                    else {
+                        const upd_res = await tryUpdateQuery(row, id);
                         if (upd_res.error) {
-                            getState().log(6, `Update child row failure ${JSON.stringify(upd_res)}`);
-                            req.flash("error", text_attr(upd_res.error));
-                            res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
-                            return;
+                            ins_upd_error = upd_res.error;
                         }
+                        trigger_return = upd_res.trigger_return;
+                    }
+                }
+                if (ins_upd_error) {
+                    await rollback();
+                    getState().log(6, `Insert or update failure ${JSON.stringify(ins_upd_error)}`);
+                    res.status(422);
+                    if (req.xhr) {
+                        res.json({ error: ins_upd_error });
                     }
                     else {
-                        const ins_res = await childTable.tryInsertRow(childRow, req.user || { role_id: 100 });
-                        if (ins_res.error) {
-                            getState().log(6, `Insert child row failure ${JSON.stringify(ins_res)}`);
-                            req.flash("error", text_attr(ins_res.error));
-                            res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
-                            return;
+                        await form.fill_fkey_options(false, optionsQuery, req.user);
+                        req.flash("error", text_attr(ins_upd_error));
+                        res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
+                    }
+                    return true;
+                }
+                for (const field of form.fields.filter((f) => f.isRepeat)) {
+                    const view_select = parse_view_select(field.metadata.view, field.metadata.relation_path);
+                    const order_field = field.metadata.order_field;
+                    const childView = View.findOne({ name: view_select.viewname });
+                    if (!childView)
+                        throw new InvalidConfiguration(`Cannot find embedded view: ${view_select.viewname}`);
+                    if (field.metadata.relation_path &&
+                        view_select.type === "RelationPath") {
+                        const targetTbl = Table.findOne({ id: childView.table_id });
+                        const relation = new Relation(field.metadata.relation_path, targetTbl.name, displayType(await childView.get_state_fields()));
+                        if (relation.type === RelationType.CHILD_LIST)
+                            updateViewSelect(view_select);
+                    }
+                    const childTable = Table.findOne({ id: field.metadata?.table_id });
+                    const submitted_row_ids = new Set((form.values[field.name] || []).map((srow) => `${srow[childTable.pk_name]}`));
+                    const childFields = new Set(childTable.fields.map((f) => f.name));
+                    let repeatIx = 0;
+                    for (const [childRow, row_ix] of form.values[field.name].map((r, ix) => [r, ix])) {
+                        // set fixed here
+                        childRow[field.metadata?.relation] = id;
+                        for (const [k, v] of Object.entries(childView?.configuration?.fixed || {})) {
+                            if (typeof childRow[k] === "undefined" &&
+                                !k.startsWith("_block_") &&
+                                childFields.has(k) &&
+                                (v || v === 0) //no nulls or empty string, but allow 0
+                            )
+                                childRow[k] = v;
+                        }
+                        if (order_field && !childRow[order_field])
+                            childRow[order_field] = row_ix;
+                        for (const file_field of field.fields.filter((f) => f.type === "File")) {
+                            const key = `${file_field.name}_${repeatIx}`;
+                            if (req.files?.[key]) {
+                                const file = await File.from_req_files(req.files[key], req.user ? req.user.id : null, (file_field.attributes &&
+                                    +file_field.attributes.min_role_read) ||
+                                    1, file_field?.attributes?.folder);
+                                childRow[file_field.name] = file.path_to_serve;
+                            }
                         }
-                        else if (ins_res.success) {
-                            submitted_row_ids.add(`${ins_res.success}`);
+                        getState().log(6, `Edit POST ready to insert/update Child row into ${childTable.name} Row=${JSON.stringify(childRow)} ID=${childRow[childTable.pk_name]} Ajax=${!!req.xhr}`);
+                        if (childRow[childTable.pk_name]) {
+                            const upd_res = await childTable.tryUpdateRow(childRow, childRow[childTable.pk_name], req.user || { role_id: 100 });
+                            if (upd_res.error) {
+                                await rollback();
+                                getState().log(6, `Update child row failure ${JSON.stringify(upd_res)}`);
+                                req.flash("error", text_attr(upd_res.error));
+                                res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
+                                return true;
+                            }
                         }
+                        else {
+                            const ins_res = await childTable.tryInsertRow(childRow, req.user || { role_id: 100 });
+                            if (ins_res.error) {
+                                await rollback();
+                                getState().log(6, `Insert child row failure ${JSON.stringify(ins_res)}`);
+                                req.flash("error", text_attr(ins_res.error));
+                                res.sendWrap(pagetitle, renderForm(form, req.csrfToken()));
+                                return true;
+                            }
+                            else if (ins_res.success) {
+                                submitted_row_ids.add(`${ins_res.success}`);
+                            }
+                        }
+                        repeatIx += 1;
                     }
-                    repeatIx += 1;
-                }
-                //need to delete any rows that are missing
-                if (originalID && field.metadata) {
-                    const childRows = getRowQuery
-                        ? await getRowQuery(field.metadata.table_id, view_select, originalID)
-                        : await childTable.getRows({
-                            [view_select.field_name]: originalID,
-                        });
-                    for (const db_child_row of childRows) {
-                        if (!submitted_row_ids.has(`${db_child_row[childTable.pk_name]}`)) {
-                            await childTable.deleteRows({
-                                [childTable.pk_name]: db_child_row[childTable.pk_name],
-                            }, req.user || { role_id: 100 });
+                    //need to delete any rows that are missing
+                    if (originalID && field.metadata) {
+                        const childRows = getRowQuery
+                            ? await getRowQuery(field.metadata.table_id, view_select, originalID)
+                            : await childTable.getRows({
+                                [view_select.field_name]: originalID,
+                            });
+                        for (const db_child_row of childRows) {
+                            if (!submitted_row_ids.has(`${db_child_row[childTable.pk_name]}`)) {
+                                await childTable.deleteRows({
+                                    [childTable.pk_name]: db_child_row[childTable.pk_name],
+                                }, req.user || { role_id: 100 });
+                            }
                         }
                     }
                 }
-            }
+            });
+            if (doReturn)
+                return;
+            //Edit-in-edit
         }
         trigger_return = trigger_return || {};
         if (trigger_return.notify && trigger_return.details)
@@ -1855,60 +1862,62 @@ module.exports = {
                 });
             }
             try {
-                if (click_action) {
-                    let container;
-                    traverseSync(layout, {
-                        container(segment) {
-                            if (segment.click_action === click_action)
-                                container = segment;
-                        },
-                    });
-                    if (!container)
-                        return { json: { error: "Action not found" } };
-                    const trigger = Trigger.findOne({ name: click_action });
-                    if (!trigger)
-                        throw new Error(`View ${name}: Container click action ${click_action} not found`);
-                    const result = await trigger.runWithoutRow({
-                        table,
-                        Table,
-                        req,
-                        row,
-                        referrer: req?.get?.("Referrer"),
-                        user: req.user,
-                    });
-                    return { json: { success: "ok", ...(result || {}) } };
-                }
-                else if (onchange_action && !rndid) {
-                    const fldCol = columns.find((c) => c.field_name === onchange_field &&
-                        c.onchange_action === onchange_action);
-                    if (!fldCol)
-                        return { json: { error: "Field not found" } };
-                    const trigger = Trigger.findOne({ name: onchange_action });
-                    if (!trigger)
-                        throw new Error(`View ${name}: On change action ${onchange_action} for field ${onchange_field} not found`);
-                    const result = await trigger.runWithoutRow({
-                        table,
-                        Table,
-                        req,
-                        row,
-                        referrer: req?.get?.("Referrer"),
-                        user: req.user,
-                    });
-                    return { json: { success: "ok", ...(result || {}) } };
-                }
-                else {
-                    const col = columns.find((c) => c.type === "Action" && c.rndid === rndid && rndid);
-                    const result = await run_action_column({
-                        col,
-                        req,
-                        table,
-                        row,
-                        res,
-                        referrer: req?.get?.("Referrer"),
-                    });
-                    //console.log("result", result);
-                    return { json: { success: "ok", ...(result || {}) } };
-                }
+                return await db.withTransaction(async () => {
+                    if (click_action) {
+                        let container;
+                        traverseSync(layout, {
+                            container(segment) {
+                                if (segment.click_action === click_action)
+                                    container = segment;
+                            },
+                        });
+                        if (!container)
+                            return { json: { error: "Action not found" } };
+                        const trigger = Trigger.findOne({ name: click_action });
+                        if (!trigger)
+                            throw new Error(`View ${name}: Container click action ${click_action} not found`);
+                        const result = await trigger.runWithoutRow({
+                            table,
+                            Table,
+                            req,
+                            row,
+                            referrer: req?.get?.("Referrer"),
+                            user: req.user,
+                        });
+                        return { json: { success: "ok", ...(result || {}) } };
+                    }
+                    else if (onchange_action && !rndid) {
+                        const fldCol = columns.find((c) => c.field_name === onchange_field &&
+                            c.onchange_action === onchange_action);
+                        if (!fldCol)
+                            return { json: { error: "Field not found" } };
+                        const trigger = Trigger.findOne({ name: onchange_action });
+                        if (!trigger)
+                            throw new Error(`View ${name}: On change action ${onchange_action} for field ${onchange_field} not found`);
+                        const result = await trigger.runWithoutRow({
+                            table,
+                            Table,
+                            req,
+                            row,
+                            referrer: req?.get?.("Referrer"),
+                            user: req.user,
+                        });
+                        return { json: { success: "ok", ...(result || {}) } };
+                    }
+                    else {
+                        const col = columns.find((c) => c.type === "Action" && c.rndid === rndid && rndid);
+                        const result = await run_action_column({
+                            col,
+                            req,
+                            table,
+                            row,
+                            res,
+                            referrer: req?.get?.("Referrer"),
+                        });
+                        //console.log("result", result);
+                        return { json: { success: "ok", ...(result || {}) } };
+                    }
+                });
             }
             catch (e) {
                 console.error(e);