@yuneta/gobj-js 7.26.15 → 7.26.16

package/dist/gobj-js.cjs.js

@@ -3297,6 +3297,20 @@ function log_stringify(v) {
 		}
 	}
 }
+function log_string(v) {
+	try {
+		return String(v);
+	} catch (e) {
+		return log_stringify(v);
+	}
+}
+function log_number(parse, v) {
+	try {
+		return parse(v);
+	} catch (e) {
+		return log_string(v);
+	}
+}
 function format_log(format) {
 	const args = Array.prototype.slice.call(arguments);
 	let output = "";
@@ -3306,18 +3320,18 @@ function format_log(format) {
 		output = format.replace(/%[sdifoO]/g, (match) => {
 			const val = args[i++];
 			switch (match) {
-				case "%s": return String(val);
+				case "%s": return log_string(val);
 				case "%d":
-				case "%i": return parseInt(val);
-				case "%f": return parseFloat(val);
+				case "%i": return log_number(parseInt, val);
+				case "%f": return log_number(parseFloat, val);
 				case "%o":
-				case "%O": return typeof val === "object" ? log_stringify(val) : String(val);
+				case "%O": return typeof val === "object" ? log_stringify(val) : log_string(val);
 				default: return match;
 			}
 		});
-		const rest = args.slice(i).map((arg) => typeof arg === "object" ? log_stringify(arg) : String(arg));
+		const rest = args.slice(i).map((arg) => typeof arg === "object" ? log_stringify(arg) : log_string(arg));
 		if (rest.length > 0) output += " " + rest.join(" ");
-	} else output = args.map((arg) => typeof arg === "object" ? log_stringify(arg) : String(arg)).join(" ");
+	} else output = args.map((arg) => typeof arg === "object" ? log_stringify(arg) : log_string(arg)).join(" ");
 	return output;
 }
 function log_error(format, ...args) {
@@ -4372,6 +4386,13 @@ function kw_get_dict(gobj, kw, path, default_value, flag) {
 			kw_set_dict_value(gobj, kw, path, default_value);
 			return default_value;
 		}
+		if (create && v === null) {
+			if (required) {
+				log_error(`path not found: '${path}'`);
+				trace_json_masked(kw);
+			}
+			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);
@@ -4436,6 +4457,13 @@ function kw_get_list(gobj, kw, path, default_value, flag) {
 			kw_set_dict_value(gobj, kw, path, default_value);
 			return default_value;
 		}
+		if (create && v === null) {
+			if (required) {
+				log_error(`path not found: '${path}'`);
+				trace_json_masked(kw);
+			}
+			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);
@@ -4649,8 +4677,8 @@ Keys:
 ["$key1", "$key2", ...]
 {"$key1":*, "$key2":*, ...}
 
-It's not a deep copy, new keys are the paths.
-If paths are empty return kw
+It's not a deep copy. A key is a KEY, not a path (as C, SDK 7.26.9).
+If keys are empty return kw
 *************************************************************/
 function kw_clone_by_keys(gobj, kw, keys, verbose) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -4677,8 +4705,8 @@ Keys:
 ["$key1", "$key2", ...]
 {"$key1":*, "$key2":*, ...}
 
-It's not a deep copy, new keys are the paths.
-If paths are empty return empty
+It's not a deep copy. A key is a KEY, not a path (as C, SDK 7.26.9).
+If keys are empty return empty
 ***************************************************************************/
 function kw_clone_by_not_keys(gobj, kw, keys, verbose) {
 	let kw_clone = {};