Keep already existing nodes when connecting two nodes

[hikemaniac]

In many of my changesets I connect newly created areas with previously existing ones. Unfortunately iD will replace a lot of older nodes with my newly created ones. This results in mass deletion and weird error messages from the editor itself (You have deleted x objects...) even though the user is pretty convinced he didn't do such a change. The older nodes are only replaced when a new node is created, but isn't yet connected to any area. When it is then dragged and connected, the older node gets replaced. Would it be possible to keep already existing nodes when connecting two of them?

[bhousel]

This is probably the same as #4674
I can change actionConnect to prefer an existing node to keep as the survivor.

iD/modules/actions/connect.js

Lines 20 to 21 in c42556e

	var last = nodeIds[nodeIds.length - 1];
	var survivor = graph.entity(last);

[hikemaniac]

You are right, the situation is different, but the core issue is the same. I'll close it if you want to.

[bhousel]

Nah ok to leave open, I'm fixing it now..