@yuneta/gobj-js 7.25.1 → 7.25.3

package/README.md

@@ -527,6 +527,14 @@ kw_get_str (gobj, kw, path, default_value, flag)
 kw_get_dict(gobj, kw, path, default_value, flag)
 kw_get_list(gobj, kw, path, default_value, flag)
 kw_get_dict_value(gobj, kw, path, default_value, flag)
+// A typed reader answers the value when it has the reader's type, and
+// the default AS GIVEN otherwise (absent key or other type), as in C:
+//   kw_get_list(gobj, {x: [1, 2]}, "x", [], 0)   → [1, 2]  (the same array)
+//   kw_get_dict(gobj, {}, "x", null, 0)          → null
+//   kw_get_bool(gobj, {x: "false"}, "x", true, 0) → true   (not a boolean: logged)
+//   kw_get_bool(gobj, {x: "false"}, "x", true, KW_WILD_NUMBER) → false
+// kw_get_bool() logs a value that is not a boolean even without
+// KW_REQUIRED ("path MUST BE a json boolean"), as the C reader does.
 
 kw_set_dict_value(gobj, kw, path, value)
 kw_set_subdict_value(gobj, kw, path, key, value)

package/dist/gobj-js.cjs.js

@@ -3215,8 +3215,8 @@ function trace_msg(format) {
 }
 function trace_json(jn, msg) {
 	if (console_wants("json", msg)) {
-		if (msg) window.console.warn("=====> " + msg);
-		window.console.dir(jn);
+		if (msg) _console.warn("=====> " + msg);
+		_console.dir(jn);
 	}
 	emit_log_callback("json", jn, current_timestamp());
 }
@@ -3604,7 +3604,14 @@ function kw_delete(gobj, kw, path) {
 	return 0;
 }
 /************************************************************
-*
+*  As the C kw_get_bool(): a boolean found is the answer, and
+*  anything else gives the default back, LOGGED ("path MUST BE a
+*  json boolean", required or not) -- unless KW_WILD_NUMBER asks
+*  to read a number (0 is false), a string ("true"/"false" in any
+*  case, else its integer) or null (false); a list or a dict is
+*  then false, logged too. Before gobj-js 7.25.2 the value went
+*  through Boolean(), so "false" was TRUE; before 7.25.3 a value
+*  of the wrong type gave the default back in silence.
 ************************************************************/
 function kw_get_bool(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3612,6 +3619,7 @@ function kw_get_bool(gobj, kw, path, default_value, flag) {
 	const required = flag && flag & kw_flag_t.KW_REQUIRED;
 	const create = flag && flag & kw_flag_t.KW_CREATE;
 	const extract = flag && flag & kw_flag_t.KW_EXTRACT;
+	const wild = flag && flag & kw_flag_t.KW_WILD_NUMBER;
 	if (b === void 0) {
 		if (create) {
 			let v = Boolean(default_value);
@@ -3623,8 +3631,25 @@ function kw_get_bool(gobj, kw, path, default_value, flag) {
 		}
 		return Boolean(default_value);
 	}
+	let value;
+	if (is_boolean(b)) value = b;
+	else if (wild && is_number(b)) value = b !== 0;
+	else if (wild && is_string(b)) {
+		if (b.toLowerCase() === "true") value = true;
+		else if (b.toLowerCase() === "false") value = false;
+		else value = (parseInt(b) || 0) !== 0;
+	} else if (wild && b === null) value = false;
+	else if (wild) {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a simple json element: '${path}'`);
+		trace_json(kw);
+		return false;
+	} else {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json boolean: '${path}'`);
+		trace_json(kw);
+		return Boolean(default_value);
+	}
 	if (extract) kw_delete(gobj, kw, path);
-	return Boolean(b);
+	return value;
 }
 /************************************************************
 *
@@ -3744,7 +3769,11 @@ function kw_get_pointer(gobj, kw, path, default_value, flag) {
 	return v;
 }
 /************************************************************
-*
+*  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 default came back as {} and a 0 as a Number object.
 ************************************************************/
 function kw_get_dict(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3753,25 +3782,25 @@ function kw_get_dict(gobj, kw, path, default_value, flag) {
 	const create = flag && flag & kw_flag_t.KW_CREATE;
 	const extract = flag && flag & kw_flag_t.KW_EXTRACT;
 	if (v === void 0) {
-		if (create) {
-			let v = Object(default_value);
-			kw_set_dict_value(gobj, kw, path, v);
-			return v;
-		} else if (required) {
+		if (create && is_object(default_value) && kw) {
+			kw_set_dict_value(gobj, kw, path, default_value);
+			return default_value;
+		}
+		if (required) {
 			log_error(`path not found: '${path}'`);
 			trace_json(kw);
 		}
-		return Object(default_value);
+		return default_value;
 	}
-	if (extract) kw_delete(gobj, kw, path);
 	if (!is_object(v)) {
 		if (required) {
-			log_error(`path value MUST BE a dict: ${path}`);
+			log_error(`${gobj_short_name(gobj)}: path value MUST BE a dict: ${path}`);
 			trace_msg(kw);
 		}
-		return Object(default_value);
+		return default_value;
 	}
-	return Object(v);
+	if (extract) kw_delete(gobj, kw, path);
+	return v;
 }
 /************************************************************
 *
@@ -3797,7 +3826,12 @@ function kw_get_dict_value(gobj, kw, path, default_value, flag) {
 	return v;
 }
 /************************************************************
-*
+*  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
+*  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].
 ************************************************************/
 function kw_get_list(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3806,25 +3840,25 @@ function kw_get_list(gobj, kw, path, default_value, flag) {
 	const create = flag && flag & kw_flag_t.KW_CREATE;
 	const extract = flag && flag & kw_flag_t.KW_EXTRACT;
 	if (v === void 0) {
-		if (create) {
-			let v = Array(default_value);
-			kw_set_dict_value(gobj, kw, path, v);
-			return v;
-		} else if (required) {
+		if (create && is_array(default_value) && kw) {
+			kw_set_dict_value(gobj, kw, path, default_value);
+			return default_value;
+		}
+		if (required) {
 			log_error(`path not found: '${path}'`);
 			trace_json(kw);
 		}
-		return Array(default_value);
+		return default_value;
 	}
-	if (extract) kw_delete(gobj, kw, path);
 	if (!is_array(v)) {
 		if (required) {
-			log_error(`path value MUST BE an array: ${path}`);
+			log_error(`${gobj_short_name(gobj)}: path value MUST BE an array: ${path}`);
 			trace_msg(kw);
 		}
-		return Array(default_value);
+		return default_value;
 	}
-	return Array(v);
+	if (extract) kw_delete(gobj, kw, path);
+	return v;
 }
 /************************************************************
 *
@@ -3860,7 +3894,7 @@ function kw_set_subdict_value(gobj, kw, path, key, value) {
 		log_error("kw is not an object");
 		return -1;
 	}
-	let subdict = kw_get_dict(gobj, kw, path, {}, true, true);
+	let subdict = kw_get_dict(gobj, kw, path, {}, kw_flag_t.KW_CREATE);
 	if (!is_object(subdict)) {
 		log_error("subdict is not an object");
 		return -1;