PropertyBinder : Minor Fixes #29

Author: PaulDemeulenaere

com.unity.visualeffectgraph/CHANGELOG.md

@@ -12,6 +12,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Avoid NullReferenceException in Previous Position Binder" component. [Case 1242351](https://issuetracker.unity3d.com/product/unity/issues/guid/1242351/)
 - Prevent from creating a context in VisualEffectSugraphOperator by draggingfrom an output slot.
 - Don't show the blocks window when context cant have blocks
+- Avoid NullReferenceException when VisualEffectAsset is null if VFXPropertyBinder [Case 1219061](https://issuetracker.unity3d.com/product/unity/issues/guid/1219061/)
+- Missing Reset function in VFXPropertyBinder [Case 1219063](https://issuetracker.unity3d.com/product/unity/issues/guid/1219063/)
 
 ## [9.0.0] - 2020-07-09
 

com.unity.visualeffectgraph/Editor/Utilities/PropertyBinding/VFXPropertyBinderEditor.cs

@@ -126,8 +126,21 @@ private class MenuPropertySetName
 
         public void CheckTypeMenu(SerializedProperty property, VFXPropertyBindingAttribute attribute, VisualEffectAsset asset)
         {
-            GenericMenu menu = new GenericMenu();
-            var parameters = (asset.GetResource().graph as UnityEditor.VFX.VFXGraph).children.OfType<UnityEditor.VFX.VFXParameter>();
+            VFXGraph graph = null;
+            if (asset != null)
+            {
+                var resource = asset.GetResource();
+                if (resource != null) //If VisualEffectGraph is store in asset bundle, we can't use this following code
+                {
+                    graph = resource.graph as VFXGraph;
+                }
+            }
+
+            if (graph == null)
+                return;
+
+            var menu = new GenericMenu();
+            var parameters = graph.children.OfType<UnityEditor.VFX.VFXParameter>();
             foreach (var param in parameters)
             {
                 string typeName = param.type.ToString();

com.unity.visualeffectgraph/Runtime/Utilities/PropertyBinding/VFXPropertyBinder.cs

@@ -36,6 +36,14 @@ private void OnEnable()
             m_VisualEffect = GetComponent<VisualEffect>();
         }
 
+        private void Reset()
+        {
+            m_VisualEffect = GetComponent<VisualEffect>();
+            m_Bindings = new List<VFXBinderBase>();
+            m_Bindings.AddRange(gameObject.GetComponents<VFXBinderBase>());
+            ClearPropertyBinders();
+        }
+
         void Update()
         {
             if (!m_ExecuteInEditor && Application.isEditor && !Application.isPlaying) return;