fix: ordering

Author: brunomikoski

Scripts/Editor/CustomEditors/CollectionCustomEditor.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.IO;
+using JetBrains.Annotations;
 using UnityEditor;
 using UnityEditor.Callbacks;
 using UnityEditor.Compilation;
@@ -151,10 +152,10 @@ protected void RemoveItemAtIndex(int selectedIndex)
         {
             SerializedProperty selectedProperty = reorderableList.serializedProperty.GetArrayElementAtIndex(selectedIndex);
             Object asset = selectedProperty.objectReferenceValue;
-            AssetDatabase.DeleteAsset(AssetDatabase.GetAssetPath(asset));
-            AssetDatabase.SaveAssets();
             reorderableList.serializedProperty.DeleteArrayElementAtIndex(selectedIndex);
             reorderableList.serializedProperty.serializedObject.ApplyModifiedProperties();
+            AssetDatabase.DeleteAsset(AssetDatabase.GetAssetPath(asset));
+            AssetDatabase.SaveAssets();
         }
 
         private void OnClickToAddNewItem(Rect buttonRect, ReorderableList list)
@@ -239,6 +240,7 @@ private void DrawCollectionItemAtIndex(Rect rect, int index, bool isActive, bool
                         AssetDatabaseUtils.RenameAsset(collectionItemSerializedProperty.objectReferenceValue, newName);
                         AssetDatabase.SaveAssets();
                     }
+                    return;
                 }
             }
 
@@ -468,7 +470,9 @@ public override void OnInspectorGUI()
             {
                 DrawSearchField();
                 DrawSynchronizeButton();
+                
                 reorderableList.DoLayoutList();
+
                 if (Event.current.type == EventType.Repaint)
                 {
                     reorderableListYPosition = GUILayoutUtility.GetLastRect().y;
@@ -507,15 +511,15 @@ protected virtual void SynchronizeAssets()
 
         private void CheckForKeyboardShortcuts()
         {
+            if (Event.current.type != EventType.KeyDown)
+                return;
+            
             if (reorderableList.index == -1)
                 return;
 
             if (!reorderableList.HasKeyboardControl())
                 return;
 
-            if (Event.current.type == EventType.Layout || Event.current.type == EventType.Repaint)
-                return;
-
             if (reorderableList.index > reorderableList.serializedProperty.arraySize - 1)
                 return;
 
@@ -650,7 +654,7 @@ private void AddNewItem()
 
                     optionsMenu.AddItem(new GUIContent($"Create New/class $NEW : {itemSubClass.Name}"), false, () =>
                     {
-                        EditorApplication.delayCall += () => { AddNewItemOfType(itemSubClass); };
+                        EditorApplication.delayCall += () => { CreateAndAddNewItemOfType(itemSubClass); };
                     });
                 }
             }
@@ -668,41 +672,6 @@ private void CreateAndAddNewItemOfType(Type itemSubClass)
                 }
             });
         }
-
-        [DidReloadScripts]
-        public static void AfterStaticAssemblyReload()
-        {
-            if (!IsWaitingForNewTypeBeCreated)
-                return;
-
-            IsWaitingForNewTypeBeCreated = false;
-
-            string lastGeneratedCollectionScriptPath =
-                CreateNewCollectionItemFromBaseWizard.LastGeneratedCollectionScriptPath.Value;
-            string lastCollectionFullName = CreateNewCollectionItemFromBaseWizard.LastCollectionFullName.Value;
-
-            if (string.IsNullOrEmpty(lastGeneratedCollectionScriptPath))
-                return;
-            
-            CreateNewCollectionItemFromBaseWizard.LastCollectionFullName.Value = string.Empty;
-            CreateNewCollectionItemFromBaseWizard.LastGeneratedCollectionScriptPath.Value = string.Empty;
-
-            string assemblyName = CompilationPipeline.GetAssemblyNameFromScriptPath(lastGeneratedCollectionScriptPath);
-
-            Type targetType = Type.GetType($"{lastCollectionFullName}, {assemblyName}");
-
-            if (CollectionsRegistry.Instance.TryGetCollectionFromItemType(targetType,
-                out ScriptableObjectCollection collection))
-            {
-                Selection.activeObject = null;
-                LAST_ADDED_COLLECTION_ITEM =  collection.AddNew(targetType);
-                
-                EditorApplication.delayCall += () =>
-                {
-                    Selection.activeObject = collection;
-                };
-            }
-        }
 
         private void AddNewItemOfType(Type targetType)
         {
@@ -989,5 +958,47 @@ private static bool EditGeneratorValidator(MenuCommand command)
             Type collectionType = command.context.GetType();
             return CollectionGenerators.GetGeneratorTypeForCollection(collectionType) != null;
         }
+
+
+        class CollectionCustomEditorAssetPostProcessor : AssetPostprocessor
+        {
+            [UsedImplicitly]
+            private static void OnPostprocessAllAssets(string[] importedAssets, string[] deletedAssets, string[] movedAssets, string[] movedFromAssetPaths, bool didDomainReload)
+            {
+                if (!didDomainReload) 
+                    return;
+                
+                if (!IsWaitingForNewTypeBeCreated)
+                    return;
+
+                IsWaitingForNewTypeBeCreated = false;
+
+                string lastGeneratedCollectionScriptPath =
+                    CreateNewCollectionItemFromBaseWizard.LastGeneratedCollectionScriptPath.Value;
+                string lastCollectionFullName = CreateNewCollectionItemFromBaseWizard.LastCollectionFullName.Value;
+
+                if (string.IsNullOrEmpty(lastGeneratedCollectionScriptPath))
+                    return;
+            
+                CreateNewCollectionItemFromBaseWizard.LastCollectionFullName.Value = string.Empty;
+                CreateNewCollectionItemFromBaseWizard.LastGeneratedCollectionScriptPath.Value = string.Empty;
+
+                string assemblyName = CompilationPipeline.GetAssemblyNameFromScriptPath(lastGeneratedCollectionScriptPath);
+
+                Type targetType = Type.GetType($"{lastCollectionFullName}, {assemblyName}");
+
+                if (CollectionsRegistry.Instance.TryGetCollectionFromItemType(targetType,
+                        out ScriptableObjectCollection collection))
+                {
+                    Selection.activeObject = null;
+                    LAST_ADDED_COLLECTION_ITEM =  collection.AddNew(targetType);
+                
+                    EditorApplication.delayCall += () =>
+                    {
+                        Selection.activeObject = collection;
+                    };
+                }
+            }
+        }
     }
 }

Scripts/Editor/EditorWindows/CreateCollectionWizard.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using JetBrains.Annotations;
 using UnityEditor;
 using UnityEditor.Callbacks;
 using UnityEditor.Compilation;
@@ -604,7 +605,7 @@ private void CreateNewCollection()
             AssetDatabase.Refresh();
 
             if (!scriptsGenerated)
-                OnAfterScriptsReloading();
+                AfterScriptsAreReady();
         }
 
         private void CreateIndirectAccess()
@@ -710,8 +711,7 @@ private bool CheckValidityOfSettings()
             return isValid;
         }
 
-        [DidReloadScripts]
-        static void OnAfterScriptsReloading()
+        private static void AfterScriptsAreReady()
         {
             if (!WaitingRecompileForContinue.Value)
                 return;
@@ -735,5 +735,19 @@ static void OnAfterScriptsReloading()
             AssetDatabase.SaveAssets();
             AssetDatabase.Refresh();
         }
+        
+        
+        class CollectionCustomEditorAssetPostProcessor : AssetPostprocessor
+        {
+            [UsedImplicitly]
+            private static void OnPostprocessAllAssets(string[] importedAssets, string[] deletedAssets, string[] movedAssets, string[] movedFromAssetPaths,
+                bool didDomainReload)
+            {
+                if (!didDomainReload)
+                    return;
+                
+                AfterScriptsAreReady();
+            }
+        }
     }
 }