@yuneta/gobj-js 7.26.13 → 7.26.15

package/dist/gobj-js.cjs.js

@@ -3281,13 +3281,20 @@ function set_remote_log_functions(remote_log_fn) {
 }
 /************************************************************
 *  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) {
@@ -4337,8 +4344,10 @@ function kw_get_pointer(gobj, kw, path, default_value, flag) {
 /************************************************************
 *  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) {
@@ -4359,8 +4368,12 @@ function kw_get_dict(gobj, kw, path, default_value, flag) {
 		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;
@@ -4394,7 +4407,9 @@ function kw_get_dict_value(gobj, kw, path, default_value, flag) {
 /************************************************************
 *  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].
@@ -4417,8 +4432,12 @@ function kw_get_list(gobj, kw, path, default_value, flag) {
 		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;
@@ -5251,7 +5270,7 @@ function jdb_update(jdb, topic_name, path, kw) {
 		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)) {
@@ -5304,7 +5323,7 @@ function jdb_delete(jdb, topic_name, path, kw) {
 		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)) {