@acorex-platform/framework-client 0.0.15 → 0.0.18

package/fesm2022/acorex-platform-framework-client-layout-builder.mjs

@@ -2676,6 +2676,8 @@ class AXPDialogRendererComponent extends AXBasePageComponent {
                 this.aggregateAndEvaluateActions();
             }, DIALOG_WIDGET_COUNT_STABLE_DELAY_MS);
         }, ...(ngDevMode ? [{ debugName: "#widgetActionsEffect" }] : /* istanbul ignore next */ []));
+        // --- Actions aggregation & evaluation ---
+        this.footerActionsEvalSeq = 0;
     }
     /**
      * Registers footer shortcuts on the platform registry at modal priority so they win over page
@@ -2845,9 +2847,7 @@ class AXPDialogRendererComponent extends AXBasePageComponent {
             },
             metadata: meta,
             patchContext: (partial) => {
-                const merged = merge({}, this.context(), partial);
-                this.context.set(merged);
-                this.layoutRenderer()?.updateContext(merged);
+                this.applyDialogContextPatch(partial);
             },
             setLoading: (loading) => this.isDialogLoading.set(loading),
         };
@@ -3012,17 +3012,49 @@ class AXPDialogRendererComponent extends AXBasePageComponent {
             close: (res) => {
                 void this.closeWithOptionalSkipValidate(res);
             },
-            context: () => this.context(),
+            context: () => {
+                const store = this.getWidgetContainer()?.contextService;
+                return (store?.data() ?? this.context());
+            },
             action: () => actionCmd,
             setLoading: (loading) => this.isDialogLoading.set(loading),
             patchContext: (partial) => {
-                const merged = merge({}, this.context(), partial);
-                this.context.set(merged);
-                this.layoutRenderer()?.updateContext(merged);
+                this.applyDialogContextPatch(partial);
             },
             invokeWidget: (widgetName, method, opts) => this.invokeWidget(widgetName, method, opts ?? {}),
         };
     }
+    /**
+     * Merges into the live widget-container store (not a possibly-stale dialog signal),
+     * and forces persisted root `id` onto the store for create-wizard related steps.
+     */
+    applyDialogContextPatch(partial) {
+        const store = this.getWidgetContainer()?.contextService;
+        const live = (store?.data() ?? this.context() ?? {});
+        const merged = merge({}, live, partial);
+        if (store) {
+            const nextId = merged['id'] ?? merged['_id'];
+            const currentId = store.data()['id'] ?? store.data()['_id'];
+            const hasNext = (typeof nextId === 'string' && nextId.length > 0) ||
+                (typeof nextId === 'number' && !Number.isNaN(nextId));
+            const hasCurrent = (typeof currentId === 'string' && currentId.length > 0) ||
+                (typeof currentId === 'number' && !Number.isNaN(currentId));
+            if (hasNext && !hasCurrent) {
+                const identity = { id: nextId };
+                if (merged['_id'] !== undefined) {
+                    identity['_id'] = merged['_id'];
+                }
+                store.patch(identity, { updateSaved: true });
+            }
+            else {
+                store.patch(partial, { updateSaved: false });
+            }
+        }
+        this.context.set(merged);
+        // Prefer identity-only sync via [context] binding + syncExternalContext identity path.
+        // Passing the full merged object after updateSaved can clearWidgets and reset the wizard.
+        this.layoutRenderer()?.updateContext(merged);
+    }
     async closeWithOptionalSkipValidate(result) {
         if (result && typeof result === 'object' && result.skipValidate) {
             this.result.emit(result);
@@ -3094,12 +3126,16 @@ class AXPDialogRendererComponent extends AXBasePageComponent {
         }
         super.close(result);
     }
-    // --- Actions aggregation & evaluation ---
     async aggregateAndEvaluateActions() {
+        const seq = ++this.footerActionsEvalSeq;
         const widgetActions = await this.collectWidgetActions();
         const dialogActions = this.collectDialogActionsFromConfig();
         const all = [...widgetActions, ...dialogActions];
         const evaluated = await this.evaluatePredicates(all);
+        // Drop stale runs (e.g. nested Assign create triggers overlapping context refreshes).
+        if (seq !== this.footerActionsEvalSeq) {
+            return;
+        }
         const visible = evaluated.filter((a) => a.hidden !== true);
         const prefix = visible.filter((a) => (a.zone ?? 'footer') === 'footer' && (a.placement ?? 'suffix') === 'prefix');
         const suffix = visible.filter((a) => (a.zone ?? 'footer') === 'footer' && (a.placement ?? 'suffix') === 'suffix');
@@ -3157,23 +3193,32 @@ class AXPDialogRendererComponent extends AXBasePageComponent {
                 dialog: { context: this.context() },
                 context: this.context(),
             };
-            const disabled = await this.evalBool(a.disabled, scope);
-            const hidden = await this.evalBool(a.hidden, scope);
+            const hiddenNeedsApi = typeof a.hidden === 'string' && /\bapi\b/.test(a.hidden);
+            const disabledNeedsApi = typeof a.disabled === 'string' && /\bapi\b/.test(a.disabled);
+            // Fail closed: missing wizard api must not reveal step-scoped footer actions (Done / Continue).
+            const disabled = disabledNeedsApi && !api ? true : await this.evalBool(a.disabled, scope, false);
+            const hidden = hiddenNeedsApi && !api ? true : await this.evalBool(a.hidden, scope, true);
             const resolvedTitle = (await this.evalActionTitle(a.title, scope)) ?? a.title;
             out.push({ ...a, disabled, hidden, title: resolvedTitle });
         }
         return out;
     }
-    async evalBool(value, scope) {
+    /**
+     * @param failClosed When evaluation throws / returns an error sentinel, use this value.
+     *                   Footer `hidden` should fail closed (`true`) so step actions do not leak.
+     */
+    async evalBool(value, scope, failClosed) {
         if (typeof value === 'boolean')
             return value;
         if (typeof value === 'string') {
             try {
                 const result = await this.expressionEvaluator.evaluate(value, scope);
+                // Expression evaluator swallows errors and returns false — treat that as failClosed
+                // only when the expression references api and api is missing (handled by caller).
                 return !!result;
             }
             catch {
-                return false;
+                return failClosed;
             }
         }
         return value;