feat: Implement Peer State Adoption for Grid Multi-Body Row Selection (#9839)

Author: tobiu

src/selection/grid/BaseModel.mjs

@@ -224,6 +224,26 @@ class BaseModel extends Model {
         me.updateRows(recordId, silent)
     }
 
+    /**
+     * Retrieves sibling Selection Models in a Multi-Body environment.
+     * @returns {Neo.selection.grid.BaseModel[]}
+     */
+    getActivePeers() {
+        let me        = this,
+            container = me.view?.gridContainer,
+            peers     = [];
+
+        if (container) {
+            peers = [
+                container.bodyStart?.selectionModel,
+                container.body?.selectionModel,
+                container.bodyEnd?.selectionModel
+            ].filter(sm => sm && sm !== me)
+        }
+
+        return peers
+    }
+
     /**
      * @param {Object} path
      * @returns {Number|String|null}
@@ -317,6 +337,22 @@ class BaseModel extends Model {
         return this.selectedRows.includes(recordId)
     }
 
+    /**
+     * @param {Neo.component.Base} component
+     */
+    register(component) {
+        super.register(component);
+
+        let me    = this,
+            peers = me.getActivePeers();
+
+        // Peer State Adoption: if siblings are already initialized, natively 
+        // adopt their state references to enforce a single state truth globally.
+        if (peers.length > 0) {
+            me.selectedRows = peers[0].selectedRows
+        }
+    }
+
     /**
      * @param {Number|String} recordId
      * @param {Boolean}       [silent=false]

src/selection/grid/RowModel.mjs

@@ -27,18 +27,20 @@ class RowModel extends BaseModel {
      *
      */
     addDomListener() {
-        let me = this;
+        let me     = this,
+            target = me.view.gridContainer || me.view.parent;
 
-        me.view.parent.on('rowClick', me.onRowClick, me)
+        target.on('rowClick', me.onRowClick, me)
     }
 
     /**
      * @param args
      */
     destroy(...args) {
-        let me = this;
+        let me     = this,
+            target = me.view.gridContainer || me.view.parent;
 
-        me.view.parent.un('rowClick', me.onRowClick, me);
+        target.un('rowClick', me.onRowClick, me);
 
         super.destroy(...args)
     }
@@ -91,14 +93,21 @@ class RowModel extends BaseModel {
     }
 
     /**
-     * @param {Object} data
+     * @param {Object} event
      */
-    onRowClick({data}) {
+    onRowClick(event) {
         let me     = this,
             {view} = me,
-            record = me.getRecord(data.path),
+            {data} = event,
+            record = event.record || me.getRecord(data.path),
             recordId;
 
+        // In a multi-body architecture, ensure we only toggle the state once per click
+        // by restricting the state mutation to the specific body that fired the event.
+        if (event.body && event.body !== view) {
+            return
+        }
+
         if (record) {
             if (me.hasAnnotations(record)) {
                 me.updateAnnotations(record)
@@ -107,6 +116,9 @@ class RowModel extends BaseModel {
 
                 me.toggleRowSelection(recordId);
 
+                // Sync visual state to sibling sub-grids
+                me.getActivePeers().forEach(peer => peer.updateRows(recordId));
+
                 view.fire(me.isSelectedRow(recordId) ? 'select' : 'deselect', {record})
             }
         }