@yuneta/gobj-js 7.26.28 → 7.27.0

package/dist/gobj-js.umd.js

@@ -2154,7 +2154,8 @@
 		if (!gobj_is_running(gobj)) {
 			if (!(gobj.gclass.gclass_flag & gclass_flag_t.gcflag_required_start_to_play)) {
 				log_warning(`${gobj_short_name(gobj)}: GObj playing without previous start`);
-				gobj_start(gobj);
+				if (start_destroys_it(gobj)) return -1;
+				if (!gobj.running) return -1;
 			} else {
 				log_error(`${gobj_short_name(gobj)}: Cannot play, start not done`);
 				return -1;
@@ -2979,12 +2980,20 @@
 	function gobj_change_parent(gobj, parent) {
 		if (gobj_is_destroying(gobj)) {
 			log_error("gobj NULL or DESTROYED");
-			return null;
+			return -1;
 		}
 		if (!(parent instanceof GObj)) {
 			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() BAD TYPE of parent`);
 			return -1;
 		}
+		if (gobj_is_destroying(parent)) {
+			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() parent NULL or DESTROYED`);
+			return -1;
+		}
+		for (let p = parent; p; p = p.parent) if (p === gobj) {
+			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() a gobj cannot be its own ancestor: ${gobj_short_name(parent)}`);
+			return -1;
+		}
 		if (gobj.parent && gobj.parent.gclass.gmt.mt_child_removed) gobj.parent.gclass.gmt.mt_child_removed(gobj.parent, gobj);
 		if (gobj.parent) {
 			_remove_child(gobj.parent, gobj);
@@ -7455,15 +7464,17 @@
 	*  The uuid of this node: ONE for the life of the page.
 	*
 	*  The one the store keeps ("host_uuid"), made and written there when
-	*  there is none. When the store cannot keep it (blocked, full, no
-	*  localStorage, outside a browser) the one this page made, warned
-	*  once. It goes in the node_uuid attribute of the yuno, the id of the
+	*  there is none. When the store cannot keep it the one this page
+	*  made: warned once when the store is blocked or full, silent when
+	*  there is no localStorage or no browser at all. It goes in the node_uuid attribute of the yuno, the id of the
 	*  identity card and the `host` of every message the client sends, so
 	*  the first answer is kept and answered again: 7.26.27 made a new
 	*  uuid on every call when the store could not keep it, and answered
-	*  null without crypto.randomUUID().
+	*  null without crypto.randomUUID(). A stored value that is not a
+	*  uuid (a foreign or corrupt write) is replaced, with a WARNING.
 	************************************************************/
 	var uuid_of_this_page = null;
+	var UUID_SHAPE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 	function node_uuid() {
 		if (uuid_of_this_page) return uuid_of_this_page;
 		const store = local_store();
@@ -7479,6 +7490,10 @@
 			uuid_of_this_page = new_uuid();
 			return uuid_of_this_page;
 		}
+		if (uuid && !UUID_SHAPE.test(uuid)) {
+			log_warning("node_uuid(): the stored host_uuid is not a uuid: a new one replaces it");
+			uuid = null;
+		}
 		if (!uuid) {
 			uuid = new_uuid();
 			try {