@yuneta/gobj-js 7.26.5 → 7.26.7

package/dist/gobj-js.cjs.js

@@ -4064,15 +4064,26 @@ function strrchr_split(str, char) {
 ************************************************************/
 function kw_delete(gobj, kw, path) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
+	const prefix = gobj ? gobj_short_name(gobj) + ": " : "";
+	if (!is_string(path)) {
+		log_error(`${prefix}kw_delete(): path must be a string: ${String(path)}`);
+		return -1;
+	}
 	let [path_, k] = strrchr_split(path, "`");
+	let v = kw;
 	if (!empty_string(k)) {
-		let v = kw_find_path(gobj, kw, path_, true);
-		if (is_array(v)) {
-			log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}kw_delete(): cannot delete an item of a list: '${path}'`);
-			return -1;
-		}
-		json_object_del(v, k);
-	} else json_object_del(kw, path);
+		v = kw_find_path(gobj, kw, path_, true);
+		if (v === void 0) return -1;
+	} else k = path;
+	if (is_array(v)) {
+		log_error(`${prefix}kw_delete(): cannot delete an item of a list: '${path}'`);
+		return -1;
+	}
+	if (!is_object(v) || !Object.prototype.hasOwnProperty.call(v, k)) {
+		log_error(`${prefix}kw_delete(): path not found: '${path}'`);
+		return -1;
+	}
+	delete v[k];
 	return 0;
 }
 /************************************************************
@@ -4424,6 +4435,10 @@ function kw_set_dict_value(gobj, kw, path, value) {
 		log_error("kw is not an object");
 		return -1;
 	}
+	if (!is_string(path)) {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}kw_set_dict_value(): path must be a string: ${String(path)}`);
+		return -1;
+	}
 	let ss = path.split("`");
 	let len = ss.length;
 	for (let i = 0; i < len; i++) {