@yuneta/gobj-js 7.25.2 → 7.25.3

package/README.md

@@ -531,8 +531,10 @@ kw_get_dict_value(gobj, kw, path, default_value, flag)
 // 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)
+//   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

@@ -3605,10 +3605,13 @@ function kw_delete(gobj, kw, path) {
 }
 /************************************************************
 *  As the C kw_get_bool(): a boolean found is the answer, and
-*  anything else gives the default back -- 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). Before gobj-js
-*  7.25.2 the value went through Boolean(), so "false" was TRUE.
+*  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`);
@@ -3636,11 +3639,13 @@ function kw_get_bool(gobj, kw, path, default_value, flag) {
 		else if (b.toLowerCase() === "false") value = false;
 		else value = (parseInt(b) || 0) !== 0;
 	} else if (wild && b === null) value = false;
-	else {
-		if (required) {
-			log_error(`${gobj_short_name(gobj)}: path value MUST BE a boolean: ${path}`);
-			trace_msg(kw);
-		}
+	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);