@yuneta/gobj-ui 7.25.9 → 7.25.11

package/src/c_yui_schema_editor.js

@@ -26,9 +26,10 @@
  *      the owner's design of M36 (2026-09-21 review) every write here
  *      raised both versions, so an edit half made was already the
  *      schema of the next start. The topic list marks what this session
- *      wrote and has not saved (the host's EV_REFRESH after a save is
- *      what forgets it) and what the host says is not saved (EV_DRAFTS,
- *      replaced whole each time it is sent).
+ *      wrote and has not saved (EV_REFRESH forgets it: the view's own
+ *      Refresh button sends it, and a host may after a save -- the
+ *      agent console's does not) and what the host says is not saved
+ *      (EV_DRAFTS, replaced whole each time it is sent).
  *
  *      WHAT ELSE IS HERE, and why each is here and not somewhere else:
  *
@@ -124,6 +125,7 @@ import {yui_tint} from "./bulma_tint.js";
 import {host_draft_ids} from "./host_drafts.js";
 
 import "./c_yui_schema_editor.css";
+import "./yui_toolbar.css";     /*  `selected_state`, the export's chosen view  */
 
 import {
     build_schema_model,
@@ -478,6 +480,60 @@ function built_on_a_replaced_model(gobj, event, kw)
     return true;
 }
 
+/***************************************************************
+ *  The reload a load refused IN session left owed (a routing
+ *  adapter's deadline on a slow `nodes`, ac_mt_command_answer()):
+ *  the model shown is kept, and it may be older than the store.
+ *  The session is up, so no reconnect will ask for it again -- it
+ *  was owed for as long as the session lasted, and a form opened
+ *  on that model wrote every field of it over the newer record
+ *  (fifth independent review; A3 of the fourth, by another path).
+ *
+ *  So the operator's NEXT action asks it. An edit -- a form, a
+ *  write, a confirmation, what reads the model whole (check,
+ *  export) -- is refused and said: it would be computed on what
+ *  the store may no longer hold, and the load that lands closes a
+ *  dialog built on the old model anyway (end_load()). A move is
+ *  not refused (reload_after_move()): it goes, and reads there.
+ *
+ *  Not on a timer, and not at once: a load refused on a deadline
+ *  asked again at once is refused the same way, and one asked on a
+ *  clock is polling. Out of session nothing is asked; the reconnect
+ *  does it (ac_transport_state()). The reload keeps this session's
+ *  draft chips: the host did not ask for it.
+ ***************************************************************/
+function owes_reload(gobj)
+{
+    let state = gobj_current_state(gobj);
+
+    return gobj.priv.reload_on_open && transport_in_session(gobj) &&
+        state !== "ST_LOADING" && state !== "ST_SAVING";
+}
+
+function reload_first(gobj, event)
+{
+    if(!owes_reload(gobj)) {
+        return false;
+    }
+    gobj.priv.reload_on_open = false;
+    log_warning(`${gobj_short_name(gobj)}: ${event} refused, the schemas shown ` +
+        `may be out of date: they are read again first`);
+    yui_shell_show_error(yui_shell_of(gobj),
+        "the schemas shown may be out of date: they are read again, try again when they are in",
+        {t: t});
+    request_model(gobj, true);
+    return true;
+}
+
+function reload_after_move(gobj)
+{
+    if(!owes_reload(gobj)) {
+        return 0;
+    }
+    gobj.priv.reload_on_open = false;
+    return request_model(gobj, true);
+}
+
 
 
 
@@ -503,11 +559,14 @@ function built_on_a_replaced_model(gobj, event, kw)
  *  and a load that LANDS closes it (end_load()): it was built on the
  *  model the load replaced, and its Save sends every field it shows.
  *  An import plan is forgotten here for the same reason: it is a list
- *  of writes computed against the model going away.
+ *  of writes computed against the model going away -- once the load
+ *  has LEFT (one that could not be sent replaces nothing).
  *
- *  `keep_written`: a load the host did not ask for -- the reconnect,
- *  a write that turned out done -- keeps what this session wrote.
- *  Only the host's EV_REFRESH (a Save sends it) forgets it.
+ *  `keep_written`: a load nobody ASKED for -- the reconnect, a write
+ *  that turned out done, the reload a refused load owed -- keeps what
+ *  this session wrote. Only EV_REFRESH forgets it: the view's own
+ *  Refresh button, or a host that sends it (after a Save, say; the
+ *  agent console does not).
  *
  *  A load that cannot leave WHOLE did not happen (out of session it
  *  is not even asked): the records the model was built on are put
@@ -526,9 +585,12 @@ function request_model(gobj, keep_written)
         return -1;      /*  not mounted on a transport yet: mt_start retries via EV_TRANSPORT_STATE  */
     }
 
-    forget_import_plan(gobj);
-
-    let previous = priv.records;
+    /*  A load asked while one is in flight (the host's EV_REFRESH is
+     *  heard in ST_LOADING) keeps the records the model SHOWN was built
+     *  on: `priv.records` is the half the first load got by then, and
+     *  put back after a failure it was a model with no treedb.  */
+    let previous = (gobj_current_state(gobj) === "ST_LOADING")?
+        priv.records_before : priv.records;
     priv.records_before = previous;
     priv.records = {treedbs: [], topics: [], cols: []};
     priv.load_error = "";
@@ -564,6 +626,9 @@ function request_model(gobj, keep_written)
         end_load(gobj);
         return -1;
     }
+    /*  Only a load that LEFT replaces the model the plan was computed
+     *  on; one that could not be sent keeps it, and the plan with it.  */
+    forget_import_plan(gobj);
     priv.pending = left;
     return 0;
 }
@@ -649,8 +714,8 @@ function build_model(gobj)
  *  Called ONLY when records arrive from the store: that is what a
  *  measurement of "changed since" can be measured against.
  *
- *  `written` is forgotten only by a load the HOST asked for (its
- *  EV_REFRESH, which a Save sends). Since M36 a write moves no
+ *  `written` is forgotten only by an EV_REFRESH (the view's own
+ *  Refresh button, or a host that sends one). Since M36 a write moves no
  *  version, so `written` is the draft chip of what this session
  *  wrote and no longer half of a version measurement: a reload of
  *  the reconnect, or of a write that turned out done, forgot the
@@ -687,10 +752,12 @@ function start_measuring(gobj)
  *  until the host saves it (C_TREEDB's save-schema, which raises the
  *  versions of what changed). Two sources say a topic is one, and
  *  they are kept APART because they are forgotten differently:
- *    - `written`: what THIS session wrote. Only a load the HOST asked
- *      for forgets it (its EV_REFRESH, which a Save sends); the
- *      reconnect's reload, the one owed after a write and a late
- *      write's keep it (start_measuring(), request_model());
+ *    - `written`: what THIS session wrote. Only EV_REFRESH forgets
+ *      it -- the view's own Refresh button sends it (in the agent
+ *      console that is the only sender), and a host may after a
+ *      Save; the reconnect's reload, the one owed after a write, a
+ *      refused load's and a late write's keep it (start_measuring(),
+ *      request_model());
  *    - `host_ids`: what the host said last (EV_DRAFTS), replaced WHOLE
  *      by the next EV_DRAFTS. Folded into `written`, a topic the host
  *      named once stayed a draft after the Save that published it.
@@ -2168,13 +2235,21 @@ function open_export(gobj, treedb)
 
     let $content = createElement2(
         ["div", {class: "SCHEMA_EXPORT box"}, [
-            ["div", {class: "SCHEMA_EXPORT_TABS tabs is-toggle mb-2"}, [
-                ["ul", {}, [
-                    ["li", {class: "SCHEMA_EXPORT_TAB is-active", "data-pane": "c"},
-                        [["a", {}, [["span", {}, "C"]]]]],
-                    ["li", {class: "SCHEMA_EXPORT_TAB", "data-pane": "json"},
-                        [["a", {}, [["span", {}, "JSON"]]]]]
-                ]]
+            /*  Two BUTTONS, not Bulma tabs: a tab's <a> with no href is
+             *  reached by no keyboard and named by nothing but its "C".
+             *  The one shown is the selected segment (`selected_state`,
+             *  yui_toolbar.css) and says so with aria-pressed.  */
+            ["div", {class: "SCHEMA_EXPORT_TABS buttons has-addons mb-2"}, [
+                ["button", {class: "SCHEMA_EXPORT_TAB button selected_state", type: "button",
+                            "data-pane": "c", "aria-pressed": "true",
+                            title: t("schema as c source"), "data-i18n-title": "schema as c source",
+                            "aria-label": t("schema as c source"), "data-i18n-aria-label": "schema as c source"},
+                    [["span", {}, "C"]]],
+                ["button", {class: "SCHEMA_EXPORT_TAB button", type: "button",
+                            "data-pane": "json", "aria-pressed": "false",
+                            title: t("schema as json"), "data-i18n-title": "schema as json",
+                            "aria-label": t("schema as json"), "data-i18n-aria-label": "schema as json"},
+                    [["span", {}, "JSON"]]]
             ]],
             ["p", {class: "SCHEMA_EXPORT_HELP help mb-2",
                    i18n: "an edit made here works and lives in no source: this is what to paste back"},
@@ -2205,7 +2280,8 @@ function open_export(gobj, treedb)
         $tab.addEventListener("click", (evt) => {
             evt.stopPropagation();
             for(let $other of $content.querySelectorAll(".SCHEMA_EXPORT_TAB")) {
-                $other.classList.toggle("is-active", $other === $tab);
+                $other.classList.toggle("selected_state", $other === $tab);
+                $other.setAttribute("aria-pressed", ($other === $tab)? "true": "false");
             }
             $text.value = ($tab.dataset.pane === "c") ? c_text : json_text;
         });
@@ -3004,7 +3080,8 @@ function ac_mt_command_answer(gobj, event, kw, src)
              *  form with it -- a form's Save then answered "Event NOT
              *  DEFINED in state ST_IDLE" (fourth independent review). It
              *  stays, as it does when the load cannot leave, and the
-             *  reload is owed.  */
+             *  reload is owed: the operator's next action asks it
+             *  (reload_first(), fifth independent review).  */
             if(priv.model) {
                 log_warning(`${gobj_short_name(gobj)}: the schemas could not be read ` +
                     `again (${priv.load_error}): the previous ones stay`);
@@ -3012,7 +3089,8 @@ function ac_mt_command_answer(gobj, event, kw, src)
                 priv.records_before = null;
                 priv.reload_on_open = true;
                 yui_shell_show_error(yui_shell_of(gobj),
-                    "cannot read the schemas again: the previous ones stay", {t: t});
+                    "cannot read the schemas again: the ones shown may be out of date, your next change reads them first",
+                    {t: t});
                 return end_load(gobj);
             }
             priv.model = null;
@@ -3307,6 +3385,28 @@ function ac_wait_for_the_load(gobj, event, kw, src)
     return -1;
 }
 
+/***************************************************************
+ *  A confirmation answered where no treedb is open: ST_EMPTY (a
+ *  reload landed on zero treedbs while it was up) or ST_IDLE (no
+ *  model). A confirmation is a shell modal, not this view's dialog,
+ *  so the load that lands does not close it, and its Yes arrives
+ *  anyway. Undeclared, it answered "Event NOT DEFINED" (fifth
+ *  independent review).
+ *
+ *  Refused with the stamp's own words: whatever it was decided on
+ *  is not on screen any more.
+ ***************************************************************/
+function ac_confirmed_on_nothing(gobj, event, kw, src)
+{
+    if(built_on_a_replaced_model(gobj, event, kw)) {
+        return -1;
+    }
+    log_warning(`${gobj_short_name(gobj)}: ${event} refused, no treedb is open`);
+    yui_shell_show_error(yui_shell_of(gobj),
+        "the schemas were read again: open the dialog again", {t: t});
+    return -1;
+}
+
 /***************************************************************
  *  {drafts: {treedb_name: [topic names]}} -- the host says which
  *  topics hold a draft in __system__ that is not saved (C_TREEDB's
@@ -3338,32 +3438,34 @@ function ac_drafts(gobj, event, kw, src)
  ***************************************************************/
 function ac_select_treedb(gobj, event, kw, src)
 {
-    return go(gobj, kw.treedb, "", false, true);
+    go(gobj, kw.treedb, "", false, true);
+    return reload_after_move(gobj);
 }
 
 function ac_select_topic(gobj, event, kw, src)
 {
-    return go(gobj, gobj.priv.treedb_id, kw.topic, false, true);
+    go(gobj, gobj.priv.treedb_id, kw.topic, false, true);
+    return reload_after_move(gobj);
 }
 
 function ac_back(gobj, event, kw, src)
 {
     let priv = gobj.priv;
 
-    if(priv.diagram) {
-        return go(gobj, priv.treedb_id, "", false, true);
-    }
-    if(priv.topic_name) {
-        return go(gobj, priv.treedb_id, "", false, true);
+    if(priv.diagram || priv.topic_name) {
+        go(gobj, priv.treedb_id, "", false, true);
+    } else {
+        go(gobj, "", "", false, true);
     }
-    return go(gobj, "", "", false, true);
+    return reload_after_move(gobj);
 }
 
 function ac_toggle_diagram(gobj, event, kw, src)
 {
     let priv = gobj.priv;
 
-    return go(gobj, priv.treedb_id, priv.topic_name, !priv.diagram, true);
+    go(gobj, priv.treedb_id, priv.topic_name, !priv.diagram, true);
+    return reload_after_move(gobj);
 }
 
 /***************************************************************
@@ -3378,11 +3480,15 @@ function ac_node_click(gobj, event, kw, src)
     if(!topic) {
         return 0;
     }
-    return go(gobj, gobj.priv.treedb_id, topic, false, true);
+    go(gobj, gobj.priv.treedb_id, topic, false, true);
+    return reload_after_move(gobj);
 }
 
 function ac_show_orphans(gobj, event, kw, src)
 {
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     open_orphans(gobj);
     return 0;
 }
@@ -3398,9 +3504,15 @@ function ac_show_orphans(gobj, event, kw, src)
 function confirm_then(gobj, message, detail, kw)
 {
     kw = Object.assign({model_gen: gobj.priv.model_gen}, kw);
+    /*  The answers are i18n KEYS: shell_modals names each button by its
+     *  label, and its defaults ("Delete", "Cancel") are not lower-case, so
+     *  no validated locale holds them -- they read English in every
+     *  language, text, title and aria-label alike.  */
     yui_shell_confirm_danger(yui_shell_of(gobj), message, {
-        t:      t,
-        detail: detail
+        t:              t,
+        detail:         detail,
+        confirm_label:  "delete",
+        cancel_label:   "cancel"
     }).then((yes) => {
         if(!yes || gobj_is_destroying(gobj)) {
             return;
@@ -3419,6 +3531,9 @@ function ac_add_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no topic open`);
         return -1;
@@ -3436,6 +3551,9 @@ function ac_edit_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!col) {
         log_error(`${gobj_short_name(gobj)}: ${event} names no column: ${kw.col}`);
         return -1;
@@ -3458,6 +3576,9 @@ function ac_duplicate_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!col) {
         log_error(`${gobj_short_name(gobj)}: ${event} names no column: ${kw.col}`);
         return -1;
@@ -3483,6 +3604,9 @@ function ac_save_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3518,6 +3642,9 @@ function ac_delete_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!col) {
         log_error(`${gobj_short_name(gobj)}: ${event} names no column: ${kw.col}`);
         return -1;
@@ -3540,6 +3667,9 @@ function ac_move_column(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no topic open`);
         return -1;
@@ -3579,6 +3709,9 @@ function ac_undo_order(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no topic open`);
         return -1;
@@ -3606,6 +3739,9 @@ function ac_add_topic(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!treedb) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no treedb open`);
         return -1;
@@ -3623,6 +3759,9 @@ function ac_edit_topic(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: ${event} names no topic: ${kw.topic}`);
         return -1;
@@ -3640,6 +3779,9 @@ function ac_save_topic(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3679,6 +3821,9 @@ function ac_delete_topic(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: ${event} names no topic: ${kw.topic}`);
         return -1;
@@ -3697,6 +3842,9 @@ function ac_delete_orphan(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3720,6 +3868,10 @@ function ac_validate(gobj, event, kw, src)
     let priv = gobj.priv;
     let treedb = current_treedb(gobj);
 
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
+
     if(!treedb) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no treedb open`);
         return -1;
@@ -3739,6 +3891,10 @@ function ac_export(gobj, event, kw, src)
 {
     let treedb = current_treedb(gobj);
 
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
+
     if(!treedb) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no treedb open`);
         return -1;
@@ -3754,6 +3910,9 @@ function ac_import(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(!treedb) {
         log_error(`${gobj_short_name(gobj)}: ${event} with no treedb open`);
         return -1;
@@ -3772,6 +3931,10 @@ function ac_preview_import(gobj, event, kw, src)
     let treedb = current_treedb(gobj);
     let incoming = null;
 
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
+
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3802,6 +3965,9 @@ function ac_apply_import(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3828,8 +3994,12 @@ function run_import(gobj)
     let priv = gobj.priv;
     let plan = priv.import_plan;
 
+    /*  Not an ERROR: a load that left forgets the plan (request_model()),
+     *  and the confirmation of it may still be up. The operator's Yes
+     *  is answered, not lost in the log.  */
     if(!plan) {
-        log_error(`${gobj_short_name(gobj)}: the import plan is gone`);
+        log_warning(`${gobj_short_name(gobj)}: the import plan is gone`);
+        yui_shell_show_error(yui_shell_of(gobj), "the import plan is gone: preview it again", {t: t});
         return -1;
     }
     close_dialog(gobj);
@@ -3855,6 +4025,9 @@ function ac_confirmed(gobj, event, kw, src)
     if(refuse_if_readonly(gobj, event)) {
         return -1;
     }
+    if(reload_first(gobj, event)) {
+        return -1;
+    }
     if(built_on_a_replaced_model(gobj, event, kw)) {
         return -1;
     }
@@ -3953,7 +4126,9 @@ function create_gclass(gclass_name)
      *  a click is worth routing through a machine.
      *---------------------------------------------*/
     const states = [
-        ["ST_IDLE", COMMON.slice()],
+        ["ST_IDLE", COMMON.concat([
+            ["EV_CONFIRMED",        ac_confirmed_on_nothing, null]
+        ])],
 
         ["ST_LOADING", COMMON.concat([
             ["EV_SAVE_COLUMN",      ac_wait_for_the_load,   null],
@@ -3964,7 +4139,9 @@ function create_gclass(gclass_name)
             ["EV_CONFIRMED",        ac_wait_for_the_load,   null]
         ])],
 
-        ["ST_EMPTY", COMMON.slice()],
+        ["ST_EMPTY", COMMON.concat([
+            ["EV_CONFIRMED",        ac_confirmed_on_nothing, null]
+        ])],
 
         ["ST_TREEDBS", COMMON.concat([
             ["EV_SELECT_TREEDB",    ac_select_treedb,       null],