Extract Way#replaceNode

js/id/actions/connect.js

@@ -20,13 +20,7 @@ iD.actions.Connect = function(nodeIds) {
             var node = graph.entity(nodeIds[i]), index;
 
             graph.parentWays(node).forEach(function (parent) {
-                while (true) {
-                    index = parent.nodes.indexOf(node.id);
-                    if (index < 0)
-                        break;
-                    parent = parent.updateNode(survivor.id, index);
-                }
-                graph = graph.replace(parent);
+                graph = graph.replace(parent.replaceNode(node.id, survivor.id));
             });
 
             graph.parentRelations(node).forEach(function (parent) {

js/id/graph/way.js

@@ -75,6 +75,19 @@ _.extend(iD.Way.prototype, {
         return this.update({nodes: nodes});
     },
 
+    replaceNode: function(needle, replacement) {
+        if (this.nodes.indexOf(needle) < 0)
+            return this;
+
+        var nodes = this.nodes.slice();
+        for (var i = 0; i < nodes.length; i++) {
+            if (nodes[i] === needle) {
+                nodes[i] = replacement;
+            }
+        }
+        return this.update({nodes: nodes});
+    },
+
     removeNode: function(id) {
         var nodes = _.without(this.nodes, id);