Minor bugfixes for GraphBehaviour and added TODOs for scene picking.

Author: TeorikDeli

Assets/Examples/Editor/GraphBehaviourEditor.cs

@@ -21,7 +21,7 @@ void OnDisable()
     public override VisualElement CreateInspectorGUI()
     {
         var root = new VisualElement();
-        var graphContainer = graphEditor.CreateInspectorGUI().Q("ExposedParameters");
+        var graphContainer = graphEditor != null ? graphEditor.CreateInspectorGUI().Q("ExposedParameters") : null;
 
         root.Add(new Button(() => EditorWindow.GetWindow<AllGraphWindow>().InitializeGraph(behaviour.graph))
         {

Assets/com.alelievr.NodeGraphProcessor/Editor/BaseGraphWindow.cs

@@ -98,7 +98,9 @@ public void InitializeGraph(BaseGraph graph)
 				AssetDatabase.SaveAssets();
 				// Unload the graph
 				graphUnloaded?.Invoke(this.graph);
-				Resources.UnloadAsset(this.graph);
+				
+				if(this.graph.name != "")	//	Only if it has a file name (GraphBehaviour graphs doesn't have one). This is a temporary solution.
+					Resources.UnloadAsset(this.graph);
 			}
 
 			graphLoaded?.Invoke(graph);

Assets/com.alelievr.NodeGraphProcessor/Editor/GraphInspector.cs

@@ -28,7 +28,7 @@ protected virtual void OnDisable()
         {
             graph.onExposedParameterListChanged -= UpdateExposedParameters;
             graph.onExposedParameterModified -= UpdateExposedParameters;
-            exposedParameterFactory.Dispose();
+            exposedParameterFactory?.Dispose(); //  Graphs that created in GraphBehaviour sometimes gives null ref.
             exposedParameterFactory = null;
         }
 

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

@@ -1162,6 +1162,7 @@ public void UpdateComputeOrder()
 
 		public void RegisterCompleteObjectUndo(string name)
 		{
+			//	TODO: GraphBehaviour graphs gives null ref when the scene is changed.
 			Undo.RegisterCompleteObjectUndo(graph, name);
 		}
 

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

@@ -709,6 +709,17 @@ protected VisualElement AddControlField(FieldInfo field, string label = null, bo
 				UpdateOtherFieldValue(field, newValue);
 			}, showInputDrawer ? "" : label);
 
+			/*if(owner.graph.name != "")
+			{
+			//	TODO: Scene Graph vs Asset Graph distinction for scene picking
+				//  If this is a graph file
+				var objectField = element.Q<ObjectField>();
+				// Check field is an ObjectField
+				if(objectField != null)
+					//  Disallow picking scene objects
+					objectField.allowSceneObjects = false;
+			}*/
+
 			if (!fieldControlsMap.TryGetValue(field, out var inputFieldList))
 				inputFieldList = fieldControlsMap[field] = new List<VisualElement>();
 			inputFieldList.Add(element);

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

@@ -67,6 +67,16 @@ public VisualElement GetParameterValueField(ExposedParameter parameter, Action<o
                 oldParameterValues[parameter] = parameter.value;
             }));
 
+            /*if(this.graph.name != "")
+            {
+			//	TODO: Scene Inspector vs Asset Inspector distinction for scene picking
+                //  If this is a graph file
+                var objectField = view.Q<ObjectField>();
+                // Check field is an ObjectField
+                if(objectField != null)
+                    //  Disallow picking scene objects
+                    objectField.allowSceneObjects = false;
+            }*/
             return view;
         }