From 6ff2f412a58a079f3aa90fa8df87602530faadc1 Mon Sep 17 00:00:00 2001
From: flare561 <flare561@gmail.com>
Date: Wed, 25 Jan 2017 03:13:45 -0600
Subject: [PATCH] Added a settings UI

---
 Rippy/AlbumData.cs                    | 11 +++
 Rippy/App.config                      | 10 +--
 Rippy/MainWindow.xaml                 | 96 +++++++++++++++++++++++++--
 Rippy/MainWindow.xaml.cs              | 50 ++++++++++----
 Rippy/Properties/Settings.Designer.cs | 10 +--
 Rippy/Properties/Settings.settings    |  8 +--
 Rippy/Rippy.csproj                    |  1 -
 Rippy/Settings.cs                     | 43 ------------
 8 files changed, 144 insertions(+), 85 deletions(-)
 delete mode 100644 Rippy/Settings.cs

diff --git a/Rippy/AlbumData.cs b/Rippy/AlbumData.cs
index d600516..9e16721 100644
--- a/Rippy/AlbumData.cs
+++ b/Rippy/AlbumData.cs
@@ -65,6 +65,17 @@ public string Medium
             set { _medium = value; OnPropertyChanged("Medium"); }
         }
 
+        public List<string> Trackers
+        {
+            get { return Rippy.Properties.Settings.Default.Trackers.Split(',').ToList(); }
+        }
+
+        public string TrackersString
+        {
+            get { return Rippy.Properties.Settings.Default.Trackers; }
+            set { Rippy.Properties.Settings.Default.Trackers = value; OnPropertyChanged("Trackers"); OnPropertyChanged("TrackersString"); }
+        }
+
         public string Tracker
         {
             get { return _tracker; }
diff --git a/Rippy/App.config b/Rippy/App.config
index 06d87c3..72ec277 100644
--- a/Rippy/App.config
+++ b/Rippy/App.config
@@ -22,14 +22,8 @@
             <setting name="TorrentDirectory" serializeAs="String">
                 <value>.\Torrents\</value>
             </setting>
-            <setting name="Trackers" serializeAs="Xml">
-                <value>
-                    <ArrayOfString xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-                        xmlns:xsd="http://www.w3.org/2001/XMLSchema">
-                        <string>https://example.com/stuff/announce</string>
-                        <string>https://example2.com/stuff2/announce</string>
-                    </ArrayOfString>
-                </value>
+            <setting name="Trackers" serializeAs="String">
+                <value>https://example1.com/stuff/announce,https://example2.com/stuff2/announce</value>
             </setting>
         </Rippy.Properties.Settings>
     </userSettings>
diff --git a/Rippy/MainWindow.xaml b/Rippy/MainWindow.xaml
index 7eb14a2..0393c75 100644
--- a/Rippy/MainWindow.xaml
+++ b/Rippy/MainWindow.xaml
@@ -34,11 +34,12 @@
         </Style>
     </Window.Resources>
     <Grid Margin="0,0,0,0">
-        <Button Panel.ZIndex="3" x:Name="settingsBtn" Content="Settings ⚙️" Style="{StaticResource LinkButton}" HorizontalAlignment="Left" Margin="440,6,0,0" VerticalAlignment="Top" Width="64" Click="settingsBtn_Click" IsEnabled="False" Visibility="Hidden"/>
+        <Button Panel.ZIndex="3" x:Name="settingsBtn" Content="Settings ⚙️" Style="{StaticResource LinkButton}" HorizontalAlignment="Left" Margin="440,6,0,0" VerticalAlignment="Top" Width="64" Click="settingsBtn_Click" />
         <StackPanel Panel.ZIndex="2" Name="pnlLeftMenu" Orientation="Horizontal" HorizontalAlignment="Left" Margin="-524,0,0,0" IsEnabled="False">
-            <Border BorderBrush="#444444" BorderThickness="1" Width="524" Background="#444444" >
+            <Border BorderBrush="#444444" BorderThickness="1" Width="524" Background="#777777" >
                 <Grid>
-                    <TextBox Name="dbPowerAmpTbx" Margin="10,37,0,0" HorizontalAlignment="Left" Width="303" Height="23" VerticalAlignment="Top" Text="{Binding Source={x:Static properties:Settings.Default},Path=DBPowerampLocation,Mode=TwoWay,UpdateSourceTrigger=PropertyChanged}">
+                    <Label Content="dBPoweramp coreconverter.exe Location" HorizontalAlignment="Left" Margin="10,10,0,0" VerticalAlignment="Top"/>
+                    <TextBox Name="dbPowerAmpTbx" Margin="10,36,0,0" HorizontalAlignment="Left" Width="303" Height="23" VerticalAlignment="Top" Text="{Binding Source={x:Static properties:Settings.Default},Path=DBPowerampLocation,Mode=TwoWay,UpdateSourceTrigger=PropertyChanged}">
                         <TextBox.ToolTip>
                             dBpoweramp coreconverter.exe Location
                         </TextBox.ToolTip>
@@ -65,8 +66,10 @@
                             </Style>
                         </TextBox.Style>
                     </TextBox>
-                    <Button x:Name="dbBrowseBtn" Content="Browse" HorizontalAlignment="Left" Margin="318,37,0,0" VerticalAlignment="Top" Width="70" Height="23"/>
-                    <TextBox Name="mp3TagTbx" Margin="10,65,0,0" HorizontalAlignment="Left" Width="303" Height="23" VerticalAlignment="Top" Text="{Binding Source={x:Static properties:Settings.Default},Path=MP3TagLocation,Mode=TwoWay,UpdateSourceTrigger=PropertyChanged}">
+                    <Button x:Name="dbBrowseBtn" Content="Browse" HorizontalAlignment="Left" Margin="318,36,0,0" VerticalAlignment="Top" Width="70" Height="23" Click="dbBrowseBtn_Click"/>
+
+                    <Label Content="Mp3tag.exe Location" HorizontalAlignment="Left" Margin="10,64,0,0" VerticalAlignment="Top"/>
+                    <TextBox Name="mp3TagTbx" Margin="10,90,0,0" HorizontalAlignment="Left" Width="303" Height="23" VerticalAlignment="Top" Text="{Binding Source={x:Static properties:Settings.Default},Path=MP3TagLocation,Mode=TwoWay,UpdateSourceTrigger=PropertyChanged}">
                         <TextBox.ToolTip>
                             Mp3tag.exe Location
                         </TextBox.ToolTip>
@@ -93,7 +96,86 @@
                             </Style>
                         </TextBox.Style>
                     </TextBox>
-                    <Button x:Name="mp3TagBrowseBtn" Content="Browse" HorizontalAlignment="Left" Margin="318,65,0,0" VerticalAlignment="Top" Width="70" Height="23"/>
+                    <Button x:Name="mp3TagBrowseBtn" Content="Browse" HorizontalAlignment="Left" Margin="318,90,0,0" VerticalAlignment="Top" Width="70" Height="23" Click="mp3TagBrowseBtn_Click"/>
+                    <Label Content="Output Directory" HorizontalAlignment="Left" Margin="10,118,0,0" VerticalAlignment="Top"/>
+                    <TextBox x:Name="outputTbx" Margin="10,144,0,0" HorizontalAlignment="Left" Width="184" Height="23" VerticalAlignment="Top" Text="{Binding OutputDirectory, Mode=TwoWay, Source={x:Static properties:Settings.Default}, UpdateSourceTrigger=PropertyChanged}">
+                        <TextBox.ToolTip>Output Directory</TextBox.ToolTip>
+                        <TextBox.Style>
+                            <Style xmlns:sys="clr-namespace:System;assembly=mscorlib" TargetType="{x:Type TextBox}">
+                                <Style.Resources>
+                                    <VisualBrush x:Key="CueBannerBrush" AlignmentX="Left" AlignmentY="Center" Stretch="None" >
+                                        <VisualBrush.Visual>
+                                            <Label Content="Output Directory" Foreground="Gray" Background="White" Width="303" Height="23"/>
+                                        </VisualBrush.Visual>
+                                    </VisualBrush>
+                                </Style.Resources>
+                                <Style.Triggers>
+                                    <Trigger Property="Text" Value="{x:Static sys:String.Empty}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="Text" Value="{x:Null}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="IsKeyboardFocused" Value="True">
+                                        <Setter Property="Background" Value="White" />
+                                    </Trigger>
+                                </Style.Triggers>
+                            </Style>
+                        </TextBox.Style>
+                    </TextBox>
+                    <Label Content="Torrent Directory" HorizontalAlignment="Left" Margin="204,118,0,0" VerticalAlignment="Top"/>
+                    <TextBox x:Name="torrentTbx" Margin="204,144,0,0" HorizontalAlignment="Left" Width="184" Height="23" VerticalAlignment="Top" Text="{Binding TorrentDirectory, Mode=TwoWay, Source={x:Static properties:Settings.Default}, UpdateSourceTrigger=PropertyChanged}">
+                        <TextBox.ToolTip>Torrent Directory</TextBox.ToolTip>
+                        <TextBox.Style>
+                            <Style xmlns:sys="clr-namespace:System;assembly=mscorlib" TargetType="{x:Type TextBox}">
+                                <Style.Resources>
+                                    <VisualBrush x:Key="CueBannerBrush" AlignmentX="Left" AlignmentY="Center" Stretch="None" >
+                                        <VisualBrush.Visual>
+                                            <Label Content="Torrent Directory" Foreground="Gray" Background="White" Width="303" Height="23"/>
+                                        </VisualBrush.Visual>
+                                    </VisualBrush>
+                                </Style.Resources>
+                                <Style.Triggers>
+                                    <Trigger Property="Text" Value="{x:Static sys:String.Empty}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="Text" Value="{x:Null}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="IsKeyboardFocused" Value="True">
+                                        <Setter Property="Background" Value="White" />
+                                    </Trigger>
+                                </Style.Triggers>
+                            </Style>
+                        </TextBox.Style>
+                    </TextBox>
+                    <Label Content="Trackers" HorizontalAlignment="Left" Margin="10,172,0,0" VerticalAlignment="Top"/>
+                    <TextBox x:Name="trackersTbx" Margin="10,203,0,0" HorizontalAlignment="Left" Width="378" Height="22" VerticalAlignment="Top" Text="{Binding TrackersString, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" TextChanged="trackersTbx_TextChanged">
+                        <TextBox.ToolTip>Trackers (Separate multiple trackers with a comma)</TextBox.ToolTip>
+                        <TextBox.Style>
+                            <Style xmlns:sys="clr-namespace:System;assembly=mscorlib" TargetType="{x:Type TextBox}">
+                                <Style.Resources>
+                                    <VisualBrush x:Key="CueBannerBrush" AlignmentX="Left" AlignmentY="Center" Stretch="None" >
+                                        <VisualBrush.Visual>
+                                            <Label Content="Trackers" Foreground="Gray" Background="White" Width="378" Height="23"/>
+                                        </VisualBrush.Visual>
+                                    </VisualBrush>
+                                </Style.Resources>
+                                <Style.Triggers>
+                                    <Trigger Property="Text" Value="{x:Static sys:String.Empty}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="Text" Value="{x:Null}">
+                                        <Setter Property="Background" Value="{StaticResource CueBannerBrush}" />
+                                    </Trigger>
+                                    <Trigger Property="IsKeyboardFocused" Value="True">
+                                        <Setter Property="Background" Value="White" />
+                                    </Trigger>
+                                </Style.Triggers>
+                            </Style>
+                        </TextBox.Style>
+                    </TextBox>
+                    <Button x:Name="saveBtn" Content="Save" HorizontalAlignment="Left" Margin="313,230,0,0" VerticalAlignment="Top" Width="75" Click="saveBtn_Click"/>
                 </Grid>
             </Border>
         </StackPanel>
@@ -296,7 +378,7 @@
                 </TextBox.Style>
             </TextBox>
             <Label Content="Tracker" HorizontalAlignment="Left" Margin="10,206,0,0" VerticalAlignment="Top" RenderTransformOrigin="0.558,2.439"/>
-            <ComboBox x:Name="trackersCobx" HorizontalAlignment="Left" Margin="10,237,0,0" VerticalAlignment="Top" Width="494" ItemsSource="{Binding Path=SettingsObj.Trackers, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType={x:Type Window}}}" SelectedItem="{Binding Path=Tracker, UpdateSourceTrigger=PropertyChanged}"/>
+            <ComboBox x:Name="trackersCobx" HorizontalAlignment="Left" Margin="10,237,0,0" VerticalAlignment="Top" Width="494" ItemsSource="{Binding Path=Trackers, UpdateSourceTrigger=PropertyChanged}" SelectedItem="{Binding Path=Tracker, UpdateSourceTrigger=PropertyChanged}"/>
             <CheckBox x:Name="mp3320Cbx" Content="MP3 320" HorizontalAlignment="Left" Margin="256,264,0,0" VerticalAlignment="Top" IsChecked="{Binding Path=MP3320, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType={x:Type Window}}}"/>
             <CheckBox x:Name="mp3V0Cbx" Content="MP3 V0" HorizontalAlignment="Left" Margin="327,264,0,0" VerticalAlignment="Top" IsChecked="{Binding Path=MP3V0, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType={x:Type Window}}}"/>
             <CheckBox x:Name="mp3V2Cbx" Content="MP3 V2" HorizontalAlignment="Left" Margin="392,264,0,0" VerticalAlignment="Top" IsChecked="{Binding Path=MP3V2, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType={x:Type Window}}}"/>
diff --git a/Rippy/MainWindow.xaml.cs b/Rippy/MainWindow.xaml.cs
index f2bd801..11fcac7 100644
--- a/Rippy/MainWindow.xaml.cs
+++ b/Rippy/MainWindow.xaml.cs
@@ -44,15 +44,13 @@ public partial class MainWindow : Window
         public bool MP3V0 { get; set; } = true;
         public bool MP3V2 { get; set; } = false;
         public bool FLAC { get; set; } = false;
-        public Settings SettingsObj { get; set; }
 
         public MainWindow()
         {
             InitializeComponent();
             _albumData = new AlbumData();
             DataContext = _albumData;
-            SettingsObj = Settings.Load();
-            _albumData.Tracker = SettingsObj.Trackers[0];
+            _albumData.Tracker = _albumData.Trackers[0];
             _createProgressDialog.DoWork += _createProgressDialog_DoWork;
             _createProgressDialog.RunWorkerCompleted += _createProgressDialog_RunWorkerCompleted;
         }
@@ -64,12 +62,12 @@ public MainWindow()
         /// <param name="name"></param>
         private void CreateFolder(string name)
         {
-            if (!Directory.Exists(SettingsObj.OutputDirectory))
-                Directory.CreateDirectory(SettingsObj.OutputDirectory);
-            if (!Directory.Exists(SettingsObj.TorrentDirectory))
-                Directory.CreateDirectory(SettingsObj.TorrentDirectory);
+            if (!Directory.Exists(Rippy.Properties.Settings.Default.OutputDirectory))
+                Directory.CreateDirectory(Rippy.Properties.Settings.Default.OutputDirectory);
+            if (!Directory.Exists(Rippy.Properties.Settings.Default.TorrentDirectory))
+                Directory.CreateDirectory(Rippy.Properties.Settings.Default.TorrentDirectory);
 
-            Directory.CreateDirectory(Path.Combine(SettingsObj.OutputDirectory, name));
+            Directory.CreateDirectory(Path.Combine(Rippy.Properties.Settings.Default.OutputDirectory, name));
         }
 
         /// <summary>
@@ -113,7 +111,7 @@ private bool TranscodeFiles(string directory, string dbargs, int totalFiles)
                 {
                     var convProc = new Process
                     {
-                        StartInfo = new ProcessStartInfo(SettingsObj.DBPowerampLocation, $"-infile=\"{infile.FullName}\" -outfile=\"{outfile.FullName}\" -convert_to\"mp3 (Lame)\" {dbargs}")
+                        StartInfo = new ProcessStartInfo(Rippy.Properties.Settings.Default.DBPowerampLocation, $"-infile=\"{infile.FullName}\" -outfile=\"{outfile.FullName}\" -convert_to\"mp3 (Lame)\" {dbargs}")
                         {
                             CreateNoWindow = true,
                             WindowStyle = ProcessWindowStyle.Hidden
@@ -139,14 +137,14 @@ private void CreateTorrent(string directory, string name)
             tc.Private = true;
             tc.Announces.Add(new List<string>() { _albumData.Tracker });
             tc.Source = "PTH";
-            tc.Create(new TorrentFileSource(directory), Path.Combine(SettingsObj.TorrentDirectory, $"{name}.torrent"));
+            tc.Create(new TorrentFileSource(directory), Path.Combine(Rippy.Properties.Settings.Default.TorrentDirectory, $"{name}.torrent"));
         }
         
 
         private bool ProcessDirectory(string name, string dbargs, int totalFiles)
         {
             CreateFolder(name);
-            var directory = Path.Combine(SettingsObj.OutputDirectory, name);
+            var directory = Path.Combine(Rippy.Properties.Settings.Default.OutputDirectory, name);
             CopyFiles(directory);
             if (!TranscodeFiles(directory, dbargs, totalFiles))
                 return false;
@@ -194,7 +192,7 @@ private void browseBtn_Click(object sender, RoutedEventArgs e)
         private async void metadataBtn_Click(object sender, RoutedEventArgs e)
         {
             mainWindow.IsEnabled = false;
-            await HelperMethods.RunProcessAsync(SettingsObj.MP3TagLocation, $"\"{_albumData.FlacFolder}\"");
+            await HelperMethods.RunProcessAsync(Rippy.Properties.Settings.Default.MP3TagLocation, $"\"{_albumData.FlacFolder}\"");
             mainWindow.IsEnabled = true;
             UpdateMetadata();
         }
@@ -252,6 +250,32 @@ private void settingsBtn_Click(object sender, RoutedEventArgs e)
                 MainWindowGrid.IsEnabled = false;
             }
         }
-        
+
+        private void dbBrowseBtn_Click(object sender, RoutedEventArgs e)
+        {
+            var dialog = new Ookii.Dialogs.Wpf.VistaOpenFileDialog();
+            dialog.FileName = _albumData.FlacFolder;
+            var result = dialog.ShowDialog();
+            Rippy.Properties.Settings.Default.DBPowerampLocation = dialog.FileName;
+        }
+
+        private void mp3TagBrowseBtn_Click(object sender, RoutedEventArgs e)
+        {
+
+            var dialog = new Ookii.Dialogs.Wpf.VistaOpenFileDialog();
+            dialog.FileName = _albumData.FlacFolder;
+            var result = dialog.ShowDialog();
+            Rippy.Properties.Settings.Default.MP3TagLocation = dialog.FileName;
+        }
+
+        private void saveBtn_Click(object sender, RoutedEventArgs e)
+        {
+            Rippy.Properties.Settings.Default.Save();
+        }
+
+        private void trackersTbx_TextChanged(object sender, TextChangedEventArgs e)
+        {
+            trackersCobx?.GetBindingExpression(ComboBox.ItemsSourceProperty)?.UpdateSource();
+        }
     }
 }
diff --git a/Rippy/Properties/Settings.Designer.cs b/Rippy/Properties/Settings.Designer.cs
index 6dbba88..f7901d5 100644
--- a/Rippy/Properties/Settings.Designer.cs
+++ b/Rippy/Properties/Settings.Designer.cs
@@ -73,14 +73,10 @@ public string TorrentDirectory {
         
         [global::System.Configuration.UserScopedSettingAttribute()]
         [global::System.Diagnostics.DebuggerNonUserCodeAttribute()]
-        [global::System.Configuration.DefaultSettingValueAttribute(@"<?xml version=""1.0"" encoding=""utf-16""?>
-<ArrayOfString xmlns:xsi=""http://www.w3.org/2001/XMLSchema-instance"" xmlns:xsd=""http://www.w3.org/2001/XMLSchema"">
-  <string>https://example.com/stuff/announce</string>
-  <string>https://example2.com/stuff2/announce</string>
-</ArrayOfString>")]
-        public global::System.Collections.Specialized.StringCollection Trackers {
+        [global::System.Configuration.DefaultSettingValueAttribute("https://example1.com/stuff/announce,https://example2.com/stuff2/announce")]
+        public string Trackers {
             get {
-                return ((global::System.Collections.Specialized.StringCollection)(this["Trackers"]));
+                return ((string)(this["Trackers"]));
             }
             set {
                 this["Trackers"] = value;
diff --git a/Rippy/Properties/Settings.settings b/Rippy/Properties/Settings.settings
index e4dba4d..d6112c4 100644
--- a/Rippy/Properties/Settings.settings
+++ b/Rippy/Properties/Settings.settings
@@ -14,12 +14,8 @@
     <Setting Name="TorrentDirectory" Type="System.String" Scope="User">
       <Value Profile="(Default)">.\Torrents\</Value>
     </Setting>
-    <Setting Name="Trackers" Type="System.Collections.Specialized.StringCollection" Scope="User">
-      <Value Profile="(Default)">&lt;?xml version="1.0" encoding="utf-16"?&gt;
-&lt;ArrayOfString xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema"&gt;
-  &lt;string&gt;https://example.com/stuff/announce&lt;/string&gt;
-  &lt;string&gt;https://example2.com/stuff2/announce&lt;/string&gt;
-&lt;/ArrayOfString&gt;</Value>
+    <Setting Name="Trackers" Type="System.String" Scope="User">
+      <Value Profile="(Default)">https://example1.com/stuff/announce,https://example2.com/stuff2/announce</Value>
     </Setting>
   </Settings>
 </SettingsFile>
\ No newline at end of file
diff --git a/Rippy/Rippy.csproj b/Rippy/Rippy.csproj
index 4253570..aac7bad 100644
--- a/Rippy/Rippy.csproj
+++ b/Rippy/Rippy.csproj
@@ -106,7 +106,6 @@
       <SubType>Designer</SubType>
     </ApplicationDefinition>
     <Compile Include="HelperMethods.cs" />
-    <Compile Include="Settings.cs" />
     <Compile Include="SourcedTorrentCreator.cs" />
     <Page Include="MainWindow.xaml">
       <Generator>MSBuild:Compile</Generator>
diff --git a/Rippy/Settings.cs b/Rippy/Settings.cs
deleted file mode 100644
index 74bc91a..0000000
--- a/Rippy/Settings.cs
+++ /dev/null
@@ -1,43 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-using System.IO;
-
-namespace Rippy
-{
-    public class Settings : AppSettings<Settings>
-    {
-        public string MP3TagLocation { get; set; }
-        public string DBPowerampLocation { get; set; }
-        public string OutputDirectory { get; set; }
-        public List<string> Trackers { get; set; }
-        public string TorrentDirectory { get; set; }
-
-        public static Settings Load()
-        {
-            if (File.Exists("settings.json"))
-            {
-                return AppSettings<Settings>.Load();
-            }
-            else
-            {
-                var defaultSettings = new Settings()
-                {
-                    MP3TagLocation = @"C:\Program Files (x86)\Mp3tag\Mp3tag.exe",
-                    DBPowerampLocation = @"C:\Program Files\dBpoweramp\coreconverter.exe",
-                    OutputDirectory = @".\Transcodes\",
-                    TorrentDirectory = @".\Torrents\",
-                    Trackers = new List<string>()
-                    {
-                        "https://example.com/stuff/announce",
-                        "https://example2.com/stuff2/announce"
-                    },
-                };
-                defaultSettings.Save();
-                return defaultSettings;
-            }
-        }
-    }
-}