[Port][6000.5][UUM-134265] Fix NullReferenceException when deleting shader input categories

Author: april-roszkowski

Packages/com.unity.shadergraph/Editor/Data/Graphs/GraphData.cs

@@ -1672,40 +1672,49 @@ public ShaderInput AddCopyOfShaderInput(ShaderInput source, int insertIndex = -1
             return copy;
         }
 
-        public void RemoveGraphInput(ShaderInput input)
+        // Removes a collection of shader inputs, deferring graph validation until all inputs have been removed
+        public void RemoveGraphInputs(IEnumerable<ShaderInput> inputs)
         {
-            switch (input)
+            foreach (ShaderInput input in inputs)
             {
-                case AbstractShaderProperty property:
-                    var propertyNodes = GetNodes<PropertyNode>().Where(x => x.property == input).ToList();
-                    foreach (var propertyNode in propertyNodes)
-                        ReplacePropertyNodeWithConcreteNodeNoValidate(propertyNode);
-                    break;
-            }
+                switch (input)
+                {
+                    case AbstractShaderProperty property:
+                        var propertyNodes = GetNodes<PropertyNode>().Where(x => x.property == input).ToList();
+                        foreach (var propertyNode in propertyNodes)
+                            ReplacePropertyNodeWithConcreteNodeNoValidate(propertyNode);
+                        break;
+                }
 
-            // Also remove this input from any category it existed in
-            foreach (var categoryData in categories)
-            {
-                if (categoryData.IsItemInCategory(input))
+                // Also remove this input from any category it existed in
+                foreach (var categoryData in categories)
                 {
-                    categoryData.RemoveItemFromCategory(input);
-                    break;
+                    if (categoryData.IsItemInCategory(input))
+                    {
+                        categoryData.RemoveItemFromCategory(input);
+                        break;
+                    }
                 }
-            }
 
-            foreach(var node in GetNodes<SubGraphNode>())
-            {
-                if (node.UsedReferenceNames().Contains(input.referenceName))
+                foreach (var node in GetNodes<SubGraphNode>())
                 {
-                    node.ValidateNode();
-                    node.Dirty(ModificationScope.Graph);
+                    if (node.UsedReferenceNames().Contains(input.referenceName))
+                    {
+                        node.ValidateNode();
+                        node.Dirty(ModificationScope.Graph);
+                    }
                 }
-            }
 
-            RemoveGraphInputNoValidate(input);
+                RemoveGraphInputNoValidate(input);
+            }
             ValidateGraph();
         }
 
+        public void RemoveGraphInput(ShaderInput input)
+        {
+            RemoveGraphInputs(new ShaderInput[] { input });
+        }
+
         public void MoveCategory(CategoryData category, int newIndex)
         {
             if (newIndex > m_CategoryData.Count || newIndex < 0)
@@ -1890,8 +1899,7 @@ public void RemoveCategory(string categoryGUID)
                 m_RemovedCategories.Add(existingCategory);
 
                 // Whenever a category is removed, also remove any inputs within that category
-                foreach (var shaderInput in existingCategory.Children)
-                    RemoveGraphInput(shaderInput);
+                RemoveGraphInputs(existingCategory.Children);
             }
             else
                 AssertHelpers.Fail("Attempted to remove a category that does not exist in the graph.");