Fix #5173: Remove Namespace.connect() calls when importing nodes from…

… any source, be it the clipboard, a prefab file or when cloning nodes from the current map. This fixes the console warnings about nodes being attached to the same namespace again. This is basically a follow-up of commit 3575385 made in 2012.

radiant/map/Map.cpp

@@ -484,9 +484,6 @@ bool Map::import(const std::string& filename)
             {
                 // Prepare our namespace for import
                 nspace->ensureNoConflicts(otherRoot);
-
-                // Now add the imported names to the local namespace
-                nspace->connect(otherRoot);
             }
 
             MergeMap(otherRoot);

radiant/selection/algorithm/Transformation.cpp

@@ -263,8 +263,6 @@ void cloneSelected(const cmd::ArgumentList& args)
     {
 		// Prepare the nodes for import
 		nspace->ensureNoConflicts(cloner.getCloneRoot());
-		// Now move all nodes into the target namespace
-		nspace->connect(cloner.getCloneRoot());
 	}
 
 	// Unselect the current selection