Added API to check if node was created from a duplication operation

Author: alelievr

Assets/com.alelievr.NodeGraphProcessor/Editor/Views/BaseGraphView.cs

@@ -222,6 +222,8 @@ void UnserializeAndPasteCallback(string operationName, string serializedData)
 
 			Dictionary<string, BaseNode> copiedNodesMap = new Dictionary<string, BaseNode>();
 
+			var unserializedGroups = data.copiedGroups.Select(g => JsonSerializer.Deserialize<Group>(g)).ToList();
+
 			foreach (var serializedNode in data.copiedNodes)
 			{
 				var node = JsonSerializer.DeserializeNode(serializedNode);
@@ -232,6 +234,8 @@ void UnserializeAndPasteCallback(string operationName, string serializedData)
 				string sourceGUID = node.GUID;
 				graph.nodesPerGUID.TryGetValue(sourceGUID, out var sourceNode);
 				//Call OnNodeCreated on the new fresh copied node
+				node.createdFromDuplication = true;
+				node.createWithinGroup = unserializedGroups.Any(g => g.innerNodeGUIDs.Contains(sourceGUID));
 				node.OnNodeCreated();
 				//And move a bit the new node
 				node.position.position += new Vector2(20, 20);
@@ -247,10 +251,8 @@ void UnserializeAndPasteCallback(string operationName, string serializedData)
 				AddToSelection(nodeViewsPerNode[node]);
 			}
 
-            foreach (var serializedGroup in data.copiedGroups)
+            foreach (var group in unserializedGroups)
             {
-                var group = JsonSerializer.Deserialize<Group>(serializedGroup);
-
                 //Same than for node
                 group.OnCreated();
 
@@ -588,7 +590,13 @@ void MouseDownCallback(MouseDownEvent e)
 		}
 
 		bool DoesSelectionContainsInspectorNodes()
-			=> selection.Any(s => s is BaseNodeView v && v.nodeTarget.needsInspector);
+		{
+			var selectedNodes = selection.Where(s => s is BaseNodeView).ToList();
+			var selectedNodesNotInInspector = selectedNodes.Except(nodeInspector.selectedNodes).ToList();
+			var nodeInInspectorWithoutSelectedNodes = nodeInspector.selectedNodes.Except(selectedNodes).ToList();
+
+			return selectedNodesNotInInspector.Any() || nodeInInspectorWithoutSelectedNodes.Any();
+		}
 
 		void DragPerformedCallback(DragPerformEvent e)
 		{

Assets/com.alelievr.NodeGraphProcessor/Runtime/Elements/BaseNode.cs

@@ -119,6 +119,16 @@ public abstract class BaseNode
 		/// </summary>
 		public virtual bool			isRenamable => false;
 
+		/// <summary>
+		/// Is the node created from a duplicate operation (either ctrl-D or copy/paste).
+		/// </summary>
+		public bool					createdFromDuplication {get; internal set; } = false;
+
+		/// <summary>
+		/// True only when the node was created from a duplicate operation and is inside a group that was also duplicated at the same time. 
+		/// </summary>
+		public bool					createWithinGroup {get; internal set; } = false;
+
 		[NonSerialized]
 		internal Dictionary< string, NodeFieldInformation >	nodeFields = new Dictionary< string, NodeFieldInformation >();
 

Assets/com.alelievr.NodeGraphProcessor/Runtime/Graph/BaseGraph.cs

@@ -174,6 +174,11 @@ protected virtual void OnEnable()
 
 		void InitializeGraphElements()
 		{
+			// Sanitize the element lists (it's possible that nodes are null if their full class name have changed)
+			// If you rename / change the assembly of a node or parameter, please use the MovedFrom() attribute to avoid breaking the graph.
+			nodes.RemoveAll(n => n == null);
+			exposedParameters.RemoveAll(e => e == null);
+
 			foreach (var node in nodes.ToList())
 			{
 				nodesPerGUID[node.GUID] = node;