pmcf 4.27.2 → 4.28.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pmcf",
-  "version": "4.27.2",
+  "version": "4.28.0",
   "publishConfig": {
     "access": "public"
   },

package/src/base.mjs

@@ -156,10 +156,24 @@ export class Base {
       this.constructor,
       (name, attribute) => attribute.owner && !attribute.type.primitive
     )) {
-      if (attribute.collection) {
-        all.push(...this[path].values());
-      } else {
-        all.push(this[path]);
+      const value = this[path];
+      if (value) {
+        if (attribute.collection) {
+          if (typeof value.values !== "function") {
+            if (value instanceof Iterator) {
+              all.push(...value);
+            } else {
+              if (typeof value === "object") {
+                //console.log("NO F", this.fullName, path, value);
+                all.push(...Object.values(value));
+              }
+            }
+          } else {
+            all.push(...value.values());
+          }
+        } else {
+          all.push(value);
+        }
       }
     }
 
@@ -222,6 +236,17 @@ export class Base {
       : name;
   }
 
+  *find(pattern) {
+    for (const node of this.walkDirections(["children"])) {
+      for (const p of pattern) {
+        if (node.fullName.match(p)) {
+          yield node;
+          break;
+        }
+      }
+    }
+  }
+
   get typeName() {
     return this.constructor.name;
   }
@@ -502,7 +527,9 @@ export class Base {
   }
 
   execFinalize() {
-    this.traverse(object => object._execFinalize());
+    for (const node of this.walkDirections(["children"])) {
+      node._execFinalize();
+    }
   }
 
   _execFinalize() {
@@ -518,24 +545,6 @@ export class Base {
     }
   }
 
-  traverse(visitor, ...args) {
-    const visited = new Set();
-    this._traverse(visited, visitor, ...args);
-    return visited;
-  }
-
-  _traverse(visited, visitor, ...args) {
-    if (visited.has(this)) {
-      return false;
-    }
-
-    visited.add(this);
-
-    visitor(this, ...args);
-
-    return true;
-  }
-
   error(...args) {
     console.error(`${this.toString()}:`, ...args);
   }

package/src/host.mjs

@@ -117,16 +117,6 @@ export class Host extends ServiceOwner {
     }
   }
 
-  _traverse(...args) {
-    if (super._traverse(...args)) {
-      for (const ni of this.networkInterfaces.values()) {
-        ni._traverse(...args);
-      }
-      return true;
-    }
-    return false;
-  }
-
   set serial(value) {
     this._serial = value;
   }

package/src/owner.mjs

@@ -23,7 +23,12 @@ export class Owner extends Base {
   static key = "name";
   static attributes = {
     networks: networks_attribute,
-    hosts: { ...default_attribute_writable, type: "host", collection: true, owner: true },
+    hosts: {
+      ...default_attribute_writable,
+      type: "host",
+      collection: true,
+      owner: true
+    },
     clusters: {
       ...default_attribute_writable,
       type: "cluster",
@@ -45,7 +50,6 @@ export class Owner extends Base {
     template: { ...boolean_attribute_writable, private: true }
   };
 
-
   static {
     addType(this);
   }
@@ -61,31 +65,6 @@ export class Owner extends Base {
     return this.template ?? super.isTemplate;
   }
 
-  _traverse(...args) {
-    if (super._traverse(...args)) {
-      for (const typeSlot of this._membersByType.values()) {
-        for (const object of typeSlot.values()) {
-          object._traverse(...args);
-        }
-      }
-
-      return true;
-    }
-
-    return false;
-  }
-
-  *find(pattern) {
-    for (const node of this.traverse(() => {})) {
-      for (const p of pattern) {
-        if (node.fullName.match(p)) {
-          yield node;
-          break;
-        }
-      }
-    }
-  }
-
   named(name) {
     if (name[0] === "/") {
       name = name.substring(this.fullName.length + 1);
@@ -195,6 +174,7 @@ export class Owner extends Base {
   }
 
   get networks() {
+    //return this._membersByType.get("network") || new Map();
     return this.typeList("network");
   }
 
@@ -234,6 +214,7 @@ export class Owner extends Base {
   }
 
   get clusters() {
+    //return this._membersByType.get("cluster") || new Map();
     return this.typeList("cluster");
   }
 

package/src/service-owner.mjs

@@ -54,17 +54,6 @@ export class ServiceOwner extends Base {
     }
   }
 
-  _traverse(...args) {
-    if (super._traverse(...args)) {
-      for (const service of this._services.values()) {
-        service._traverse(...args);
-      }
-
-      return true;
-    }
-    return false;
-  }
-
   typeNamed(typeName, name) {
     if (typeName === "service") {
       const service = this.services.get(name);

package/src/services/bind.mjs

@@ -407,6 +407,7 @@ export class BindService extends ExtraSourceService {
       ...default_attribute_writable,
       type: bind_group,
       collection: true,
+      owner: true,
       writable: true
     },
     primaries: {
@@ -501,17 +502,6 @@ export class BindService extends ExtraSourceService {
     }
   }
 
-  _traverse(...args) {
-    if (super._traverse(...args)) {
-      for (const group of Object.values(this.groups)) {
-        group._traverse(...args);
-      }
-      return true;
-    }
-
-    return false;
-  }
-
   typeNamed(type, name) {
     if (type === bind_group.name) {
       return this.groups[name];

package/src/subnet.mjs

@@ -28,7 +28,6 @@ export class Subnet extends Base {
     family: string_attribute
   };
 
-
   static {
     addType(this);
   }
@@ -84,17 +83,6 @@ export class Subnet extends Base {
   get longAddress() {
     return `${this.longPrefix}/${this.prefixLength}`;
   }
-
-  _traverse(...args) {
-    if (super._traverse(...args)) {
-      for (const network of this.networks) {
-        network._traverse(...args);
-      }
-      return true;
-    }
-
-    return false;
-  }
 }
 
 const _owner = { addObject() {} };