@vectoriox/iox-builder 4.18.0 → 4.18.1

package/fesm2022/vectoriox-iox-builder.mjs

@@ -2751,6 +2751,12 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "20.3.31", ngImpo
  * meant only the toolbar could ever drive it. The toolbar's own switcher UI broke the header
  * (`.form-state-bar` had no CSS rule — see forms-and-inputs-plan.md Part 2) and was, separately,
  * the wrong home per board 3: the switcher belongs to the Form, not to global chrome.
+ *
+ * `setTarget` takes the real node, not just its id, and keeps it (privately) for `setState` to
+ * use — `OverlayService.getNodeRef` matches by REFERENCE, not id (a repeater clone shares its
+ * template's id), so a caller re-deriving "the current form" from an id alone could resolve the
+ * wrong instance. Every consumer of this service only ever needs the id for display; the node
+ * itself is this service's own business.
  */
 class FormStateService {
     constructor(_overlay) {
@@ -2760,28 +2766,33 @@ class FormStateService {
             state: FormRenderState.Idle,
         });
         this.state$ = this._state$.asObservable();
+        this._targetNode = null;
     }
     getSnapshot() {
         return this._state$.value;
     }
     /** Called whenever the selection changes. A different form (or none) resets to Idle — a
      *  visitor never lands on a form already showing someone else's Sent/Failed block. */
-    setTarget(formId) {
-        const current = this._state$.value;
-        if (current.formId === formId)
+    setTarget(form) {
+        const formId = form?.id ?? null;
+        if (this._state$.value.formId === formId)
             return;
+        this._targetNode = form;
         this._state$.next({ formId, state: FormRenderState.Idle });
     }
     /**
-     * Writes the state straight onto the live component instance — a VIEW state, never page
-     * data, so it must never be persisted (a saved page must not ship showing its own success
-     * message). Two input names on purpose: the canvas renders the container twin (`formState`)
-     * while the engine renders `IoxFormComponent` (`renderState`); `safeSetInput` drops whichever
-     * the rendered component doesn't declare, so this works on either path unbranched.
+     * Writes the state straight onto the live component instance for the CURRENT target (set via
+     * `setTarget`) — a VIEW state, never page data, so it must never be persisted (a saved page
+     * must not ship showing its own success message). Two input names on purpose: the canvas
+     * renders the container twin (`formState`) while the engine renders `IoxFormComponent`
+     * (`renderState`); `safeSetInput` drops whichever the rendered component doesn't declare, so
+     * this works on either path unbranched. A no-op when nothing is in scope.
      */
-    setState(form, state) {
-        this._state$.next({ formId: form.id ?? null, state });
-        const ref = this._overlay.getNodeRef(form)?.componentRef;
+    setState(state) {
+        if (!this._targetNode)
+            return;
+        this._state$.next({ formId: this._targetNode.id ?? null, state });
+        const ref = this._overlay.getNodeRef(this._targetNode)?.componentRef;
         if (ref) {
             safeSetInput(ref, 'formState', state);
             safeSetInput(ref, 'renderState', state);
@@ -5971,7 +5982,7 @@ class BuilderComponent {
     }
     set selectedPath(path) {
         this._selectedPath = path;
-        this.formStateService.setTarget(nearestFormNode(path)?.id ?? null);
+        this.formStateService.setTarget(nearestFormNode(path));
     }
     // Type-safe event forwarding for toolbar outputs
     handleToolbarModeChange(mode) {