@yuneta/gobj-ui 7.23.192 → 7.23.193

package/src/yui_row_search.test.js

@@ -110,3 +110,25 @@ describe("is_hook_size", () => {
         expect(is_hook_size(null)).toBe(false);
     });
 });
+
+/*
+ *  The children of a hook as the NODE EVENTS deliver them: `{id,
+ *  topic_name}`, with no hook_name. Not taken for a reference, their
+ *  topic_name -- the same word on every row -- was a wildcard for every
+ *  row that had received an UPDATED (M30 of the 2026-09-21 review).
+ */
+describe("a hook's children as the node events deliver them", () => {
+    const row = {id: "w1", devices: [{id: "d7", topic_name: "devices"}]};
+
+    it("are references: only their id is looked at", () => {
+        expect(is_fkey_ref({id: "d7", topic_name: "devices"})).toBe(true);
+        expect(row_matches(row, "d7")).toBe(true);
+        expect(row_matches(row, "devices")).toBe(false);
+    });
+
+    it("an object that carries more than a reference is data, searched whole", () => {
+        const data = {id: "x", topic_name: "devices", note: "blue"};
+        expect(is_fkey_ref(data)).toBe(false);
+        expect(row_matches({id: "r", payload: data}, "blue")).toBe(true);
+    });
+});

package/src/yui_table_select.js

@@ -94,6 +94,54 @@ function yui_selected_rows(tabulator)
     }
 }
 
+/***************************************************************
+ *      The IDENTITY of some rows: the value of the table's `index`
+ *      field (Tabulator's default is `id`). This is what crosses a
+ *      confirmation dialog -- never the rows' POSITIONS, which the
+ *      table regenerates with every addData / deleteRow while the
+ *      dialog is open, so a position names whatever row sits there
+ *      when the person answers.
+ ***************************************************************/
+function yui_row_ids(tabulator, rows)
+{
+    let field = (tabulator && tabulator.options && tabulator.options.index) || "id";
+    let ids = [];
+    for(let row of (rows || [])) {
+        let id = row ? row[field] : undefined;
+        if(id !== undefined && id !== null && id !== "") {
+            ids.push(id);
+        }
+    }
+    return ids;
+}
+
+/***************************************************************
+ *      The rows that carry these ids NOW -> {rows, missing}.
+ *      `rows` are the rows' data; `missing` the ids no row carries
+ *      any more. A missing id is never replaced by another row.
+ ***************************************************************/
+function yui_rows_by_ids(tabulator, ids)
+{
+    let rows = [];
+    let missing = [];
+    for(let id of (ids || [])) {
+        let row = null;
+        if(tabulator && typeof tabulator.getRow === "function") {
+            try {
+                row = tabulator.getRow(id) || null;
+            } catch(e) {
+                row = null;     /*  the table is gone  */
+            }
+        }
+        if(row) {
+            rows.push(row.getData());
+        } else {
+            missing.push(id);
+        }
+    }
+    return {rows: rows, missing: missing};
+}
+
 /***************************************************************
  *      Drop the selection (after acting on it, or after a
  *      reload that leaves the ticked rows behind).
@@ -240,6 +288,8 @@ export {
     yui_selection_column,
     yui_selection_settings,
     yui_selected_rows,
+    yui_row_ids,
+    yui_rows_by_ids,
     yui_clear_selection,
     yui_wire_selection,
     yui_selection_bar,

package/src/yui_table_select.test.js

@@ -12,6 +12,8 @@ import {
     yui_selection_settings,
     yui_selected_rows,
     yui_clear_selection,
+    yui_row_ids,
+    yui_rows_by_ids,
 } from "./yui_table_select.js";
 
 describe("the column", () => {
@@ -52,3 +54,55 @@ describe("reading and dropping the selection", () => {
             .not.toThrow();
     });
 });
+
+/*
+ *  A table as the delete of the treedb topic table meets it AFTER a
+ *  confirmation: rows came and went while the dialog was open. A position
+ *  names whatever row sits there now; an id names the row the question named
+ *  (the regression of d60ec78, A6 of the 2026-09-21 review).
+ */
+function fake_table(rows, index)
+{
+    return {
+        options: index ? {index: index} : {},
+        getRow(id) {
+            let data = rows.find((r) => r[index || "id"] === id);
+            return data ? {getData: () => data} : false;
+        }
+    };
+}
+
+describe("row identity across a confirmation", () => {
+    test("the ids are read from the table's index field", () => {
+        let table = fake_table([], "name");
+        expect(yui_row_ids(table, [{name: "a"}, {name: "b"}])).toEqual(["a", "b"]);
+    });
+
+    test("with no index configured the field is `id`, Tabulator's default", () => {
+        expect(yui_row_ids(fake_table([]), [{id: "x"}])).toEqual(["x"]);
+    });
+
+    test("a row with no id has no identity to carry", () => {
+        expect(yui_row_ids(fake_table([]), [{id: ""}, {}, null, {id: "k"}])).toEqual(["k"]);
+    });
+
+    test("the rows are the ones that carry the ids NOW, not the ones at a position", () => {
+        /*  The question named "b". Before the answer, "a" went away: at
+         *  position 2 there is now "c", and a position would delete it.  */
+        let table = fake_table([{id: "b"}, {id: "c"}]);
+        let found = yui_rows_by_ids(table, ["b"]);
+        expect(found.rows).toEqual([{id: "b"}]);
+        expect(found.missing).toEqual([]);
+    });
+
+    test("an id that is gone is reported, never replaced by another row", () => {
+        let table = fake_table([{id: "c"}]);
+        let found = yui_rows_by_ids(table, ["b", "c"]);
+        expect(found.rows).toEqual([{id: "c"}]);
+        expect(found.missing).toEqual(["b"]);
+    });
+
+    test("no table: nothing found, everything missing", () => {
+        expect(yui_rows_by_ids(null, ["b"])).toEqual({rows: [], missing: ["b"]});
+    });
+});