Merge pull request #358 from jetelain/settings-async

Settings async
tgjones · Jul 30, 2023 · 3099ce3 · 3099ce3
2 parents 779c5dc + 9706108
commit 3099ce3
Show file tree

Hide file tree

Showing 5 changed files with 53 additions and 16 deletions.
diff --git a/src/Gemini/Modules/Settings/ISettingsEditor.cs b/src/Gemini/Modules/Settings/ISettingsEditor.cs
@@ -1,4 +1,4 @@
-namespace Gemini.Modules.Settings
+namespace Gemini.Modules.Settings
 {
     public interface ISettingsEditor
     {
@@ -7,4 +7,4 @@ public interface ISettingsEditor
 
         void ApplyChanges();
     }
-}
+}
diff --git a/src/Gemini/Modules/Settings/ISettingsEditorAsync.cs b/src/Gemini/Modules/Settings/ISettingsEditorAsync.cs
@@ -0,0 +1,12 @@
+using System.Threading.Tasks;
+
+namespace Gemini.Modules.Settings
+{
+    public interface ISettingsEditorAsync
+    {
+        string SettingsPageName { get; }
+        string SettingsPagePath { get; }
+
+        Task ApplyChangesAsync();
+    }
+}
diff --git a/src/Gemini/Modules/Settings/ViewModels/SettingsEditorWrapper.cs b/src/Gemini/Modules/Settings/ViewModels/SettingsEditorWrapper.cs
@@ -0,0 +1,26 @@
+using System.Threading.Tasks;
+
+namespace Gemini.Modules.Settings.ViewModels
+{
+    internal sealed class SettingsEditorWrapper : ISettingsEditorAsync
+    {
+        private readonly ISettingsEditor _editor;
+
+        public SettingsEditorWrapper(ISettingsEditor editor)
+        {
+            _editor = editor;
+        }
+
+        public string SettingsPageName => _editor.SettingsPageName;
+
+        public string SettingsPagePath => _editor.SettingsPagePath;
+
+        public ISettingsEditor ViewModel => _editor;
+
+        public Task ApplyChangesAsync()
+        {
+            _editor.ApplyChanges();
+            return Task.CompletedTask;
+        }
+    }
+}
diff --git a/src/Gemini/Modules/Settings/ViewModels/SettingsPageViewModel.cs b/src/Gemini/Modules/Settings/ViewModels/SettingsPageViewModel.cs
@@ -7,11 +7,13 @@ public class SettingsPageViewModel
         public SettingsPageViewModel()
         {
             Children = new List<SettingsPageViewModel>();
-            Editors = new List<ISettingsEditor>();
+            Editors = new List<object>();
         }
 
         public string Name { get; set; }
-        public List<ISettingsEditor> Editors { get; private set; }
-        public List<SettingsPageViewModel> Children { get; internal set; }
+
+        public List<object> Editors { get; } // ISettingsEditor or ISettingsEditorAsync
+
+        public List<SettingsPageViewModel> Children { get; }
     }
 }
diff --git a/src/Gemini/Modules/Settings/ViewModels/SettingsViewModel.cs b/src/Gemini/Modules/Settings/ViewModels/SettingsViewModel.cs
@@ -4,7 +4,6 @@
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
-using System.Windows.Input;
 using Caliburn.Micro;
 using Gemini.Framework;
 using Gemini.Properties;
@@ -15,7 +14,7 @@ namespace Gemini.Modules.Settings.ViewModels
     [PartCreationPolicy (CreationPolicy.NonShared)]
     public class SettingsViewModel : WindowBase
     {
-        private IEnumerable<ISettingsEditor> _settingsEditors;
+        private IEnumerable<ISettingsEditorAsync> _settingsEditors;
         private SettingsPageViewModel _selectedPage;
 
         public SettingsViewModel()
@@ -40,7 +39,8 @@ protected override async Task OnInitializeAsync(CancellationToken cancellationTo
             await base.OnInitializeAsync(cancellationToken);
 
             var pages = new List<SettingsPageViewModel>();
-            _settingsEditors = IoC.GetAll<ISettingsEditor>();
+
+            _settingsEditors = IoC.GetAll<ISettingsEditorAsync>().Concat(IoC.GetAll<ISettingsEditor>().Select(e => new SettingsEditorWrapper(e)));
 
             foreach (var settingsEditor in _settingsEditors)
             {
@@ -50,14 +50,11 @@ protected override async Task OnInitializeAsync(CancellationToken cancellationTo
 
                 if (page == null)
                 {
-                    page = new SettingsPageViewModel
-                    {
-                        Name = settingsEditor.SettingsPageName,
-                    };
+                    page = new SettingsPageViewModel { Name = settingsEditor.SettingsPageName };
                     parentCollection.Add(page);
                 }
 
-                page.Editors.Add(settingsEditor);
+                page.Editors.Add(settingsEditor is SettingsEditorWrapper wrapper ? (object)wrapper.ViewModel : (object)settingsEditor);
             }
 
             Pages = pages;
@@ -76,7 +73,7 @@ private static SettingsPageViewModel GetFirstLeafPageRecursive(List<SettingsPage
             return GetFirstLeafPageRecursive(firstPage.Children);
         }
 
-        private List<SettingsPageViewModel> GetParentCollection(ISettingsEditor settingsEditor,
+        private List<SettingsPageViewModel> GetParentCollection(ISettingsEditorAsync settingsEditor,
             List<SettingsPageViewModel> pages)
         {
             if (string.IsNullOrEmpty(settingsEditor.SettingsPagePath))
@@ -92,7 +89,7 @@ private List<SettingsPageViewModel> GetParentCollection(ISettingsEditor settings
 
                 if (page == null)
                 {
-                    page = new SettingsPageViewModel {Name = pathElement};
+                    page = new SettingsPageViewModel { Name = pathElement };
                     pages.Add(page);
                 }
 
@@ -106,7 +103,7 @@ public async Task SaveChanges()
         {
             foreach (var settingsEditor in _settingsEditors)
             {
-                settingsEditor.ApplyChanges();
+                await settingsEditor.ApplyChangesAsync();
             }
 
             await TryCloseAsync(true);