@yuneta/gobj-ui 7.23.43 → 7.23.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.23.43",
+  "version": "7.23.44",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",

package/src/c_yui_form.js

@@ -71,6 +71,7 @@ import "tabulator-tables/dist/css/tabulator.min.css"; // Import Tabulator CSS
 import "tabulator-tables/dist/css/tabulator_bulma.css";
 
 import "./tabulator.css";
+import {is_time_field} from "./json_view_helpers.js";
 import { TabulatorFull as Tabulator } from "tabulator-tables"; // Import Full Tabulator JS
 // import { Tabulator } from "tabulator-tables";  // Import light Tabulator JS
 
@@ -1365,12 +1366,12 @@ function create_form_field(
                     onChange: function() {
                         gobj_send_event(gobj, "EV_RECORD_CHANGED", {}, gobj);
                     },
+                    /*  La lista vive en json_view_helpers.js: el visor de
+                     *  JSON y este editor tienen que reconocer los MISMOS
+                     *  campos, y tener cada uno su copia es como dejaron de
+                     *  hacerlo.  */
                     timestampTag: function({field, value, path}) {
-                        return field === '__t__' || field === '__tm__' ||
-                            field === 'tm' || field === 't' || field === 'time' ||
-                            field === 'from_t' || field === 'to_t' ||
-                            field === 't_input' || field === 't_output' ||
-                            field === 'from_tm' || field === 'to_tm';
+                        return is_time_field(field);
                     },
                 }
             });

package/src/json_view_helpers.js

@@ -183,18 +183,34 @@ export function subtree_matches(value, term)
 
 /************************************************************
  *   Fields whose numeric value is an epoch timestamp.
- *   Mirrors the timestampTag list wired into C_YUI_FORM's
- *   jsoneditor control, so the two viewers agree.
+ *
+ *   THE ONE LIST.  C_YUI_FORM's jsoneditor takes its
+ *   `timestampTag` from here: it used to carry its own copy,
+ *   and a copy is how the two stopped agreeing.
+ *
+ *   TWO SPELLINGS, AND LAS DOS SON DEL BACKEND.  timeranger2
+ *   writes `from_t`/`from_tm` in the match_cond of a query and
+ *   `fr_t`/`fr_tm` in the metadata of a topic and of a file --
+ *   which is what a Raw JSON of a treedb actually shows.  With
+ *   only the long pair listed, `to_t` got its date and the
+ *   `fr_t` right above it did not, which reads as a broken
+ *   annotator rather than a missing name.
  ************************************************************/
+const TIME_FIELDS = [
+    "__t__", "__tm__",
+    "t", "tm", "time",
+    "from_t", "to_t", "from_tm", "to_tm",
+    "fr_t", "fr_tm",
+    "t_input", "t_output"
+];
+
 export function is_time_field(field)
 {
-    return field === "__t__" || field === "__tm__" ||
-        field === "tm" || field === "t" || field === "time" ||
-        field === "from_t" || field === "to_t" ||
-        field === "t_input" || field === "t_output" ||
-        field === "from_tm" || field === "to_tm";
+    return TIME_FIELDS.indexOf(field) >= 0;
 }
 
+export {TIME_FIELDS};
+
 /************************************************************
  *   Format an epoch value as a local wall-clock string.
  *   Accepts seconds or milliseconds; returns null when the

package/src/json_view_helpers.test.js

@@ -310,3 +310,36 @@ test("is_pure_collection: the schema this came from", () => {
     /*  A column has content, so it keeps its card. */
     expect(is_pure_collection(col)).toBe(false);
 });
+
+
+/*
+ *  DOS ORTOGRAFIAS, Y LAS DOS SON DEL BACKEND.
+ *
+ *  timeranger2 escribe `from_t`/`from_tm` en el match_cond de una consulta y
+ *  `fr_t`/`fr_tm` en los metadatos de un topic y de un fichero -- que es lo
+ *  que enseña el Raw JSON de un treedb.  Con solo la larga en la lista,
+ *  `to_t` salia fechado y el `fr_t` de encima no: se lee como un anotador
+ *  roto y es un nombre que falta.
+ */
+test("is_time_field reconoce la abreviada, la que sale en un Raw JSON", () => {
+    expect(is_time_field("fr_t")).toBe(true);
+    expect(is_time_field("fr_tm")).toBe(true);
+});
+
+test("is_time_field sigue reconociendo la larga, la de las consultas", () => {
+    expect(is_time_field("from_t")).toBe(true);
+    expect(is_time_field("from_tm")).toBe(true);
+});
+
+test("is_time_field: las dos mitades de cada par, o la fila queda a medias", () => {
+    [["fr_t", "to_t"], ["fr_tm", "to_tm"],
+     ["from_t", "to_t"], ["from_tm", "to_tm"]].forEach((pair) => {
+        expect(is_time_field(pair[0])).toBe(is_time_field(pair[1]));
+    });
+});
+
+test("is_time_field no fecha lo que no es una fecha", () => {
+    ["rows", "id", "", undefined, null].forEach((f) => {
+        expect(is_time_field(f)).toBe(false);
+    });
+});