@akcelik/strct 3.1.0 → 3.1.1

package/fesm2022/akcelik-strct.mjs

@@ -11603,12 +11603,18 @@ class StrctDatagrid {
     somePageSelected = computed(() => !this.allPageSelected() &&
         this.selectionRows().some((r) => this.selected().has(this.idOf(r))), /* @ts-ignore */
     ...(ngDevMode ? [{ debugName: "somePageSelected" }] : /* istanbul ignore next */ []));
-    /** Resolve a row's stable identity (defaults to the row object itself). */
+    /** Resolve a row's stable identity: its `rowId` value, or the row object
+     *  itself when there is no `rowId` or it resolves to null/undefined. */
     idOf(row) {
         const id = this.rowId();
         if (id == null)
             return row;
-        return typeof id === 'function' ? id(row) : row[id];
+        const resolved = typeof id === 'function' ? id(row) : row[id];
+        // An unresolvable id must not collapse rows onto one another: a Set holds
+        // `undefined` once, so every row missing the field would share one
+        // identity — select one, select them all. `??`, not `||`: 0 and '' are
+        // legitimate ids.
+        return resolved ?? row;
     }
     rowKey(row) {
         return this.idOf(row);