Call recreateCopy after recusion test not before. #52

Author: iTris666

com.unity.visualeffectgraph/CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Fixed
 - Fix for node window staying when clicking elsewhere
 - Prefab highlight missing for initial event name toggle [Case 1263012](https://issuetracker.unity3d.com/product/unity/issues/guid/1263012/)
+- fix crash when creating a loop in subgraph operators [Case 1251523](https://issuetracker.unity3d.com/product/unity/issues/guid/1251523/)
 
 ## [8.2.0] - 2020-07-08
 

com.unity.visualeffectgraph/Editor/Models/Blocks/VFXSubgraphBlock.cs

@@ -196,16 +196,6 @@ public void PatchInputExpressions()
             VFXSubgraphUtility.TransferExpressionToParameters(inputExpressions, GetParameters(t => VFXSubgraphUtility.InputPredicate(t)).OrderBy(t => t.order));
         }
 
-        protected override void OnInvalidate(VFXModel model, InvalidationCause cause)
-        {
-            if (cause == InvalidationCause.kSettingChanged && (subgraph != null || object.ReferenceEquals(m_Subgraph, null)))
-            {
-                RecreateCopy();
-            }
-
-            base.OnInvalidate(model, cause);
-        }
-
         public VFXModel[] subChildren
         {
             get { return m_SubChildren; }
@@ -272,11 +262,11 @@ protected internal override void Invalidate(VFXModel model, InvalidationCause ca
                 if (graph != null && subgraph != null && m_Subgraph.GetResource() != null)
                 {
                     var otherGraph = m_Subgraph.GetResource().GetOrCreateGraph();
-                    if (otherGraph != m_UsedSubgraph)
-                        RecreateCopy();
                     if (otherGraph == graph || otherGraph.subgraphDependencies.Contains(graph.GetResource().visualEffectObject))
                         m_Subgraph = null; // prevent cyclic dependencies.
-                    if (graph.GetResource().isSubgraph) // BuildSubgraphDependenciesis called for vfx by recompilation, but in subgraph we must call it explicitely
+                    if (otherGraph != m_UsedSubgraph)
+                        RecreateCopy();
+                    if (graph.GetResource().isSubgraph) // BuildSubgraphDependencies is called for vfx by recompilation, but in subgraph we must call it explicitely
                         graph.BuildSubgraphDependencies();
                 }
                 else if (m_UsedSubgraph != null)