@woosh/meep-engine 2.46.13 → 2.46.15

package/package.json

@@ -4,7 +4,7 @@
   "productName": "Meep",
   "description": "production-ready JavaScript game engine based on Entity Component System Architecture",
   "author": "Alexander Goldring",
-  "version": "2.46.13",
+  "version": "2.46.15",
   "main": "build/meep.module.js",
   "module": "build/meep.module.js",
   "scripts": {

package/src/core/model/node-graph/NodeGraph.js

@@ -84,7 +84,48 @@ export class NodeGraph {
 
         this.reset();
 
-        const other_nodes = other.nodes.asArray();
+        this.merge(other);
+
+    }
+
+    /**
+     *
+     * @returns {NodeGraph}
+     */
+    clone() {
+        const r = new NodeGraph();
+
+        r.copy(this);
+
+        return r;
+    }
+
+    /**
+     * Merge another graph into this one
+     * Supplied graph does not change as a result
+     * @param {NodeGraph} other
+     */
+    merge(other) {
+        if (other === this) {
+            // can't merge with self, operation is pointless
+            return;
+        }
+
+        this.mergeFragment({
+            nodes: other.nodes.asArray(),
+            connections: other.connections.asArray()
+        });
+    }
+
+    /**
+     * Merge foreign nodes and associated connections into this graph
+     * New node instances and connections will be created to reflect these inside this graph
+     * @param {NodeInstance[]} nodes
+     * @param {Connection[]} [connections]
+     */
+    mergeFragment({ nodes, connections = [] }) {
+
+        const other_nodes = nodes;
         const node_count = other_nodes.length;
 
         /**
@@ -99,7 +140,7 @@ export class NodeGraph {
         }
 
         // create connections
-        const other_connections = other.connections.asArray();
+        const other_connections = connections;
         const connection_count = other_connections.length;
 
         for (let i = 0; i < connection_count; i++) {
@@ -203,6 +244,32 @@ export class NodeGraph {
         return result;
     }
 
+    /**
+     *
+     * @param {Type<NodeDescription>} Klass
+     * @returns {NodeInstance[]}
+     */
+    getNodesByDescriptionClass(Klass) {
+
+        assert.defined(Klass, 'Klass');
+        assert.notNull(Klass, 'Klass');
+
+        const result = [];
+
+        const nodes = this.nodes;
+        const n = nodes.length;
+
+        for (let i = 0; i < n; i++) {
+            const node = nodes.get(i);
+
+            if (node.description.constructor === Klass) {
+                result.push(node);
+            }
+        }
+
+        return result;
+    }
+
     /**
      *
      * @param {number} id

package/src/core/model/node-graph/util/graph_clone_by_node_subset.js

@@ -18,39 +18,13 @@ export function graph_clone_by_node_subset({ graph, nodes }) {
 
     const result = new NodeGraph();
 
-    /**
-     * Maps from source node IDs to result node IDs
-     * @type {Object<number,number>}
-     */
-    const cloned_nodes = {};
+    const connections = graph_collect_connections_amongst_nodes({ graph, nodes });
 
-    nodes.forEach(n => {
-
-        if (!graph.hasNode(n)) {
-            throw new Error(`Referenced node ${n} is not a found in the graph`);
-        }
-
-        const id = result.createNode(n.description);
-
-        cloned_nodes[n.id] = id;
+    result.mergeFragment({
+        nodes,
+        connections
     });
 
-    graph_collect_connections_amongst_nodes({ graph, nodes })
-        .forEach(connection => {
-
-            const source_node_id = cloned_nodes[connection.source.instance.id];
-            const source_endpoint = result.getNode(source_node_id).getEndpoint(connection.source.port.id);
-
-            const target_node_id = cloned_nodes[connection.target.instance.id];
-            const target_endpoint = result.getNode(target_node_id).getEndpoint(connection.target.port.id);
-
-            result.createConnection(
-                source_node_id, source_endpoint.id,
-                target_node_id, target_endpoint.id
-            );
-
-        });
-
     return result;
 
 }