From 55fc8d5e4019b7ff24099b3e4deeb0a6672ac523 Mon Sep 17 00:00:00 2001
From: Matt Nadareski <mnadareski@outlook.com>
Date: Tue, 29 Jun 2021 22:54:50 -0700
Subject: [PATCH] Populate themes to its own helper method

---
 RedBookPlayer/GUI/SettingsWindow.xaml.cs | 54 +++++++++++++++---------
 1 file changed, 33 insertions(+), 21 deletions(-)

diff --git a/RedBookPlayer/GUI/SettingsWindow.xaml.cs b/RedBookPlayer/GUI/SettingsWindow.xaml.cs
index f88a3ac..cfe293e 100644
--- a/RedBookPlayer/GUI/SettingsWindow.xaml.cs
+++ b/RedBookPlayer/GUI/SettingsWindow.xaml.cs
@@ -44,30 +44,42 @@ namespace RedBookPlayer.GUI
         void InitializeComponent()
         {
             AvaloniaXamlLoader.Load(this);
-
-            _themeList                  =  this.FindControl<ListBox>("ThemeList");
-            _themeList.SelectionChanged += ThemeList_SelectionChanged;
-
-            List<string> items = new List<string>();
-            items.Add("default");
-
-            if(Directory.Exists("themes/"))
-            {
-                foreach(string dir in Directory.EnumerateDirectories("themes/"))
-                {
-                    string themeName = dir.Split('/')[1];
-
-                    if (!File.Exists($"themes/{themeName}/view.xaml"))
-                        continue;
-
-                    items.Add(themeName);
-                }
-            }
-
-            _themeList.Items = items;
+            
+            PopulateThemes();
 
             this.FindControl<Button>("ApplyButton").Click            += ApplySettings;
             this.FindControl<Slider>("VolumeSlider").PropertyChanged += (s, e) => UpdateView();
         }
+
+        /// <summary>
+        /// Populate the list of themes
+        /// </summary>
+        private void PopulateThemes()
+        {
+            // Get a reference to the theme list
+            _themeList = this.FindControl<ListBox>("ThemeList");
+            _themeList.SelectionChanged += ThemeList_SelectionChanged;
+
+            // Create a list of all found themes
+            List<string> items = new List<string>();
+            items.Add("default");
+
+            // Ensure the theme directory exists
+            if(!Directory.Exists("themes/"))
+                Directory.CreateDirectory("themes/");
+
+            // Add all theme directories if they're valid
+            foreach(string dir in Directory.EnumerateDirectories("themes/"))
+            {
+                string themeName = dir.Split('/')[1];
+
+                if(!File.Exists($"themes/{themeName}/view.xaml"))
+                    continue;
+
+                items.Add(themeName);
+            }
+
+            _themeList.Items = items;
+        }
     }
 }
\ No newline at end of file