Merge branch 'master' into refactor-serialization-system

# Conflicts:
#	Assets/com.alelievr.NodeGraphProcessor/package.json
#	CHANGELOG.md

Author: alelievr

Assets/Examples/DefaultNodes/Nodes/OutputNode.cs

@@ -0,0 +1,21 @@
+using System.Collections;
+using System.Collections.Generic;
+using UnityEngine;
+using GraphProcessor;
+using System.Linq;
+
+[System.Serializable, NodeMenuItem("Custom/OutputNode")]
+public class OutputNode : BaseNode
+{
+	[Input(name = "In")]
+    public float                input;
+
+	public override string		name => "OutputNode";
+
+	public override bool		deletable => false;
+
+	protected override void Process()
+	{
+		// Do stuff
+	}
+}

Assets/Examples/DefaultNodes/Nodes/OutputNode.cs.meta

@@ -70,6 +70,9 @@ public void Initialize(BaseGraphView owner, BaseNode node)
 			nodeTarget = node;
 			this.owner = owner;
 
+			if (!node.deletable)
+				capabilities &= ~Capabilities.Deletable;
+
 			owner.computeOrderUpdated += ComputeOrderUpdatedCallback;
 			node.onMessageAdded += AddMessageView;
 			node.onMessageRemoved += RemoveMessageView;
@@ -561,7 +564,13 @@ protected void AddInputContainer()
 
 		protected virtual void DrawDefaultInspector(bool fromInspector = false)
 		{
-			var fields = nodeTarget.GetType().GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.DeclaredOnly);
+			var fields = nodeTarget.GetType().GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance)
+				// Filter fields from the BaseNode type since we are only interested in user-defined fields
+				// (better than BindingFlags.DeclaredOnly because we keep any inherited user-defined fields) 
+				.Where(f => f.DeclaringType != typeof(BaseNode))
+				// Order by MetadataToken to sync the order with the port order (make sure FieldDrawers are next to the correct port)
+				//TODO: Also consider custom port order
+				.OrderBy(f => f.MetadataToken);
 
 			foreach (var field in fields)
 			{
@@ -735,6 +744,11 @@ protected VisualElement AddControlField(FieldInfo field, string label = null, bo
 					controlsContainer.Add(element);
 				}
 			}
+			else
+			{
+				// Make sure we create an empty placeholder if FieldFactory can not provide a drawer
+				if (showInputDrawer) AddEmptyField(field, false);
+			}
 
 			var visibleCondition = field.GetCustomAttribute(typeof(VisibleIf)) as VisibleIf;
 			if (visibleCondition != null)
@@ -979,6 +993,8 @@ void UpdatePortViewWithPorts(NodePortContainer ports, List< PortView > portViews
 					SyncPortCounts(ports, new PortView[]{});
 				else if (ports.Count == 0) // Same when there is no ports
 					SyncPortCounts(new NodePort[]{}, portViews);
+				else if (portViews.Count != ports.Count)
+					SyncPortCounts(ports, portViews);
 				else
 				{
 					var p = ports.GroupBy(n => n.fieldName);

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

@@ -47,6 +47,9 @@ public abstract class BaseNode
 		/// <summary>Show the node controlContainer only when the mouse is over the node</summary>
 		public virtual bool			showControlsOnHover => false;
 
+		/// <summary>True if the node can be deleted, false otherwise</summary>
+		public virtual bool			deletable => true;
+
 		/// <summary>
 		/// Container of input ports
 		/// </summary>

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

@@ -16,9 +16,9 @@ public static void Call(Action a)
             }
             catch (Exception e)
             {
-                Debug.LogError(e);
+                Debug.LogException(e);
             }
 #endif
         }
     }
-}
+}

Assets/com.alelievr.NodeGraphProcessor/Runtime/Utils/ExceptionToLog.cs

@@ -12,6 +12,17 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Change
 - Serialization backend (use [SerializeReference] instead of JSON)
 
+## [0.10.4]
+
+### Added
+- Added deletable bool API on the BaseNode
+
+### Fixed
+- Fix missing and inconsistent field drawers on inherited node fields
+- Fix inconsistent field drawer positions after unsupported fields
+- Fixed port sync code
+- Fixed exception to log not handling correctly the stacktraces
+
 ## [0.10.1]
 
 ### Fixed