@woosh/meep-engine 2.46.14 → 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.14",
+  "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++) {
@@ -123,18 +164,6 @@ export class NodeGraph {
 
     }
 
-    /**
-     *
-     * @returns {NodeGraph}
-     */
-    clone() {
-        const r = new NodeGraph();
-
-        r.copy(this);
-
-        return r;
-    }
-
     /**
      *
      * @param {function(NodeInstance):*} visitor

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;
 
 }