LayoutViewerにもGroup並び順を反映

Author: shairo

Assets/SmartAddresser/Editor/Core/Tools/Addresser/LayoutRuleEditor/LayoutRuleEditorPresenter.cs

@@ -112,7 +112,7 @@ private void SetupActiveView(LayoutRuleData data)
             _assetSaveService.SetAsset(data);
 
             var groups = addressableAssetSettings.groups;
-            
+
 #if AAS_SORT_ORDER
             var orderSettings = AddressableAssetGroupSortSettings.GetSettings(addressableAssetSettings);
             groups = addressableAssetSettings.groups

Assets/SmartAddresser/Editor/Core/Tools/Addresser/LayoutViewer/LayoutViewerPresenter.cs

@@ -9,6 +9,8 @@
 using SmartAddresser.Editor.Foundation.TinyRx;
 using SmartAddresser.Editor.Foundation.TinyRx.ObservableProperty;
 using UnityEditor;
+using UnityEditor.AddressableAssets;
+using UnityEditor.AddressableAssets.Settings.GroupSchemas;
 using UnityEngine;
 using UnityEngine.Assertions;
 
@@ -47,13 +49,6 @@ public void Dispose()
             _editingData.Dispose();
         }
 
-        private void AddGroupView(Group group, bool reload = true)
-        {
-            _view.TreeView.AddGroup(group);
-            if (reload)
-                _view.TreeView.Reload();
-        }
-
         public void SetupView(ILayoutRuleDataRepository dataRepository)
         {
             _setupViewDisposables.Clear();
@@ -84,9 +79,7 @@ private void SetupActiveView(LayoutRuleData data)
             layout.Validate(false, validationSettings.DuplicateAddresses, validationSettings.DuplicateAssetPaths,
                 validationSettings.EntryHasMultipleVersions);
             _layout = layout;
-            foreach (var group in layout.Groups)
-                AddGroupView(group);
-            _view.TreeView.Reload();
+            ApplyGroupsToTreeView();
             _view.ActiveMode.Value = LayoutViewerView.Mode.Viewer;
             _view.ActiveAssetName.Value = data.name;
 
@@ -185,15 +178,32 @@ void OnRefreshButtonClicked()
                     validationSettings.EntryHasMultipleVersions);
                 _layout = layout;
 
-                _view.TreeView.ClearItems();
-                foreach (var group in layout.Groups)
-                    AddGroupView(group);
-                _view.TreeView.Reload();
+                ApplyGroupsToTreeView();
             }
 
             #endregion
         }
 
+        public void ApplyGroupsToTreeView()
+        {
+            _view.TreeView.ClearItems();
+
+            if (_layout == null)
+            {
+                return;
+            }
+
+            var addressableAssetSettings = AddressableAssetSettingsDefaultObject.Settings;
+            var orderSettings = AddressableAssetGroupSortSettings.GetSettings(addressableAssetSettings);
+            foreach (var group in _layout.Groups.OrderBy(g =>
+                         Array.IndexOf(orderSettings.sortOrder, g.AddressableGroup.Guid)))
+            {
+                _view.TreeView.AddGroup(group);
+            }
+
+            _view.TreeView.Reload();
+        }
+
         private void CleanupViewEventHandlers()
         {
             _view.TreeView.OnSelectionChanged -= OnTreeViewSelectionChanged;

Assets/SmartAddresser/Editor/Core/Tools/Addresser/LayoutViewer/LayoutViewerWindow.cs

@@ -5,6 +5,8 @@
 using SmartAddresser.Editor.Foundation.EditorSplitView;
 using SmartAddresser.Editor.Foundation.TinyRx;
 using UnityEditor;
+using UnityEditor.AddressableAssets;
+using UnityEditor.AddressableAssets.Settings;
 using UnityEngine;
 
 namespace SmartAddresser.Editor.Core.Tools.Addresser.LayoutViewer
@@ -21,14 +23,28 @@ internal sealed class LayoutViewerWindow : EditorWindow
         private LayoutViewerPresenter _presenter;
         private LayoutViewerView _view;
 
+        private AddressableAssetSettings _addressableAssetSettings;
+
         private void OnEnable()
         {
             minSize = new Vector2(600, 200);
             Setup();
+            
+            _addressableAssetSettings = AddressableAssetSettingsDefaultObject.Settings;
+            if (_addressableAssetSettings)
+            {
+                _addressableAssetSettings.OnModification += OnAddressableAssetSettingsModified;
+            }
         }
 
         private void OnDisable()
         {
+            if (_addressableAssetSettings)
+            {
+                _addressableAssetSettings.OnModification -= OnAddressableAssetSettingsModified;
+                _addressableAssetSettings = null;
+            }
+            
             _presenter?.Dispose();
             _view?.Dispose();
         }
@@ -66,5 +82,25 @@ public static void Open()
         {
             GetWindow<LayoutViewerWindow>(WindowName);
         }
+
+        private void OnAddressableAssetSettingsModified(
+            AddressableAssetSettings settings,
+            AddressableAssetSettings.ModificationEvent e,
+            object obj)
+        {
+            // AssetGroupの並び順反映
+            if (e == AddressableAssetSettings.ModificationEvent.GroupMoved)
+            {
+                _presenter.ApplyGroupsToTreeView();
+            }
+            
+            // コールバックを末尾に登録しなおす
+            // AssetGroupの並び順はOnModificationコールバック内でシリアライズされるので、それより後に更新しないと反映できないため
+            if (_addressableAssetSettings)
+            {
+                _addressableAssetSettings.OnModification -= OnAddressableAssetSettingsModified;
+                _addressableAssetSettings.OnModification += OnAddressableAssetSettingsModified;
+            }
+        }
     }
 }