From aea80870d337c5d42260105f88038bc3882f026b Mon Sep 17 00:00:00 2001
From: Mark McDowall <markus.mcd5@gmail.com>
Date: Mon, 13 May 2013 22:22:51 -0700
Subject: [PATCH] Enum Config values retrieved properly

---
 NzbDrone.Core/Configuration/ConfigService.cs | 31 +++++++++++++-------
 1 file changed, 20 insertions(+), 11 deletions(-)

diff --git a/NzbDrone.Core/Configuration/ConfigService.cs b/NzbDrone.Core/Configuration/ConfigService.cs
index 4f6b33494..80d66df74 100644
--- a/NzbDrone.Core/Configuration/ConfigService.cs
+++ b/NzbDrone.Core/Configuration/ConfigService.cs
@@ -87,16 +87,16 @@ namespace NzbDrone.Core.Configuration
 
         public SabPriorityType SabBacklogTvPriority
         {
-            get { return (SabPriorityType)GetValueInt("SabBacklogTvPriority"); }
+            get { return GetValueEnum("SabBacklogTvPriority", SabPriorityType.Default); }
 
-            set { SetValue("SabBacklogTvPriority", (int)value); }
+            set { SetValue("SabBacklogTvPriority", value); }
         }
 
         public SabPriorityType SabRecentTvPriority
         {
-            get { return (SabPriorityType)GetValueInt("SabRecentTvPriority"); }
+            get { return GetValueEnum("SabRecentTvPriority", SabPriorityType.Default); }
 
-            set { SetValue("SabRecentTvPriority", (int)value); }
+            set { SetValue("SabRecentTvPriority", value); }
         }
 
         public String DownloadClientTvDirectory
@@ -275,9 +275,9 @@ namespace NzbDrone.Core.Configuration
 
         public DownloadClientType DownloadClient
         {
-            get { return (DownloadClientType)GetValueInt("DownloadClient"); }
+            get { return GetValueEnum("DownloadClientType", DownloadClientType.Sabnzbd); }
 
-            set { SetValue("DownloadClient", (int)value); }
+            set { SetValue("DownloadClient", value); }
         }
 
         public string BlackholeDirectory
@@ -347,7 +347,6 @@ namespace NzbDrone.Core.Configuration
             set { SetValue("RssSyncInterval", value); }
         }
 
-
         public Boolean IgnoreArticlesWhenSortingSeries
         {
             get { return GetValueBoolean("IgnoreArticlesWhenSortingSeries", true); }
@@ -399,16 +398,16 @@ namespace NzbDrone.Core.Configuration
 
         public PriorityType NzbgetBacklogTvPriority
         {
-            get { return (PriorityType)GetValueInt("NzbgetBacklogTvPriority"); }
+            get { return GetValueEnum("NzbgetBacklogTvPriority", PriorityType.Normal); }
 
-            set { SetValue("NzbgetBacklogTvPriority", (int)value); }
+            set { SetValue("NzbgetBacklogTvPriority", value); }
         }
 
         public PriorityType NzbgetRecentTvPriority
         {
-            get { return (PriorityType)GetValueInt("NzbgetRecentTvPriority"); }
+            get { return GetValueEnum("NzbgetRecentTvPriority", PriorityType.Normal); }
 
-            set { SetValue("NzbgetRecentTvPriority", (int)value); }
+            set { SetValue("NzbgetRecentTvPriority", value); }
         }
 
         public string NzbRestrictions
@@ -438,6 +437,11 @@ namespace NzbDrone.Core.Configuration
             return Convert.ToInt32(GetValue(key, defaultValue));
         }
 
+        public T GetValueEnum<T>(string key, T defaultValue)
+        {
+            return (T)Enum.Parse(typeof(T), GetValue(key, defaultValue), true);
+        }
+
         public string GetValue(string key, object defaultValue, bool persist = false)
         {
             EnsureCache();
@@ -493,6 +497,11 @@ namespace NzbDrone.Core.Configuration
             ClearCache();
         }
 
+        public void SetValue(string key, Enum value)
+        {
+            SetValue(key, value.ToString().ToLower());
+        }
+
         public void SaveValues(Dictionary<string, object> configValues)
         {
             var allWithDefaults = AllWithDefaults();