@yuneta/gobj-js 7.26.13 → 7.26.15

package/dist/gobj-js.umd.js

@@ -3284,13 +3284,20 @@
 	}
 	/************************************************************
 	*  JSON.stringify for a log line: a circular object (a gobj,
-	*  a DOM node) must not make the log call itself throw.
+	*  a DOM node), a BigInt or a toJSON() that throws must not make
+	*  the log call itself throw -- and neither must the fallback:
+	*  String() throws too, on an object with no prototype or with a
+	*  toString() that throws.
 	************************************************************/
 	function log_stringify(v) {
 		try {
 			return JSON.stringify(v);
 		} catch (e) {
-			return String(v);
+			try {
+				return String(v);
+			} catch (e2) {
+				return `<${typeof v}: not printable>`;
+			}
 		}
 	}
 	function format_log(format) {
@@ -4340,8 +4347,10 @@
 	/************************************************************
 	*  As the C kw_get_dict(): a dict found is the answer, and
 	*  anything else gives the default back AS GIVEN (null stays
-	*  null). KW_CREATE stores a dict default, never a null one.
-	*  Before gobj-js 7.25.2 the answer went through Object(), so a
+	*  null). KW_CREATE stores a dict default, never a null one, and
+	*  replaces a null at the path; over another type it logs and
+	*  gives the default back unwritten (C returns NULL there, SDK
+	*  7.26.9). Before gobj-js 7.25.2 the answer went through Object(), so a
 	*  null default came back as {} and a 0 as a Number object.
 	************************************************************/
 	function kw_get_dict(gobj, kw, path, default_value, flag) {
@@ -4362,8 +4371,12 @@
 			return default_value;
 		}
 		if (!is_object(v)) {
-			if (required) {
-				log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json dict: '${path}'`);
+			if (create && v === null && is_object(default_value)) {
+				kw_set_dict_value(gobj, kw, path, default_value);
+				return default_value;
+			}
+			if (required || create) {
+				log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json dict: '${path}'` + (create ? ", cannot create" : ""));
 				trace_json_masked(kw);
 			}
 			return default_value;
@@ -4397,7 +4410,9 @@
 	/************************************************************
 	*  As the C kw_get_list(): a list found is the answer, and
 	*  anything else gives the default back AS GIVEN. KW_CREATE
-	*  stores a list default, never a null one. Before gobj-js
+	*  stores a list default, never a null one, and replaces a null
+	*  at the path; over another type it logs and gives the default
+	*  back unwritten (C returns NULL there, SDK 7.26.9). Before gobj-js
 	*  7.25.2 the answer went through Array(), which WRAPS its
 	*  argument: a list found came back as [list], a default of []
 	*  as [[]] and a null one as [null].
@@ -4420,8 +4435,12 @@
 			return default_value;
 		}
 		if (!is_array(v)) {
-			if (required) {
-				log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json list: '${path}'`);
+			if (create && v === null && is_array(default_value)) {
+				kw_set_dict_value(gobj, kw, path, default_value);
+				return default_value;
+			}
+			if (required || create) {
+				log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json list: '${path}'` + (create ? ", cannot create" : ""));
 				trace_json_masked(kw);
 			}
 			return default_value;
@@ -5254,7 +5273,7 @@
 			return null;
 		}
 		if (!is_object(kw) || empty_string(kw["id"])) {
-			log_error(`jdb_update: record without id: ${JSON.stringify(kw)}`);
+			log_error(`jdb_update: record without id: ${log_stringify(kw)}`);
 			return null;
 		}
 		if (!is_string(path)) {
@@ -5307,7 +5326,7 @@
 			return null;
 		}
 		if (!is_object(kw) || empty_string(kw["id"])) {
-			log_error(`jdb_delete: record without id: ${JSON.stringify(kw)}`);
+			log_error(`jdb_delete: record without id: ${log_stringify(kw)}`);
 			return null;
 		}
 		if (!is_string(path)) {