From a350be339c6e035ff082fb505b1ff974a03de8d9 Mon Sep 17 00:00:00 2001
From: echel0n <justin.tabish@gmail.com>
Date: Fri, 23 May 2014 09:35:01 -0700
Subject: [PATCH] Fix for issue causing issues loading and saving search
 providers

---
 .../interfaces/default/config_providers.tmpl  |  2 +-
 sickbeard/__init__.py                         |  4 +-
 sickbeard/providers/__init__.py               |  1 -
 sickbeard/webserve.py                         | 47 ++++---------------
 4 files changed, 13 insertions(+), 41 deletions(-)

diff --git a/gui/slick/interfaces/default/config_providers.tmpl b/gui/slick/interfaces/default/config_providers.tmpl
index 7521c75e..a8f82cf0 100644
--- a/gui/slick/interfaces/default/config_providers.tmpl
+++ b/gui/slick/interfaces/default/config_providers.tmpl
@@ -225,7 +225,7 @@ var show_nzb_providers = #if $sickbeard.USE_NZBS then "true" else "false"#;
                         </div>
                         #end if
 
-                        #if $hasattr($curNewznabProvider, 'backlog_only'):
+                        #if $hasattr($curNzbProvider, 'backlog_only'):
                         <div class="field-pair">
                             <input type="checkbox" name="${curNzbProvider.getID()}_backlog_only" id="${curNzbProvider.getID()}_backlog_only" #if $curNzbProvider.backlog_only then "checked=\"checked\"" else ""#/>
                             <label class="clearfix" for="${curNzbProvider.getID()}_backlog_only">
diff --git a/sickbeard/__init__.py b/sickbeard/__init__.py
index 49e2282f..42a1c6eb 100644
--- a/sickbeard/__init__.py
+++ b/sickbeard/__init__.py
@@ -867,8 +867,8 @@ def initialize(consoleLogging=True):
         NEWZNAB_DATA = check_setting_str(CFG, 'Newznab', 'newznab_data', '')
         newznabProviderList = providers.getNewznabProviderList(NEWZNAB_DATA)
 
-        torrentRssData = check_setting_str(CFG, 'TorrentRss', 'torrentrss_data', '')
-        torrentRssProviderList = providers.getTorrentRssProviderList(torrentRssData)
+        TORRENTRSS_DATA = check_setting_str(CFG, 'TorrentRss', 'torrentrss_data', '')
+        torrentRssProviderList = providers.getTorrentRssProviderList(TORRENTRSS_DATA)
 
         if not os.path.isfile(CONFIG_FILE):
             logger.log(u"Unable to find '" + CONFIG_FILE + "', all settings will be default!", logger.DEBUG)
diff --git a/sickbeard/providers/__init__.py b/sickbeard/providers/__init__.py
index 818bdeae..b8579492 100644
--- a/sickbeard/providers/__init__.py
+++ b/sickbeard/providers/__init__.py
@@ -58,7 +58,6 @@ def sortedProviderList():
 
     return newList
 
-
 def makeProviderList():
     return [x.provider for x in [getProviderModule(y) for y in __all__] if x]
 
diff --git a/sickbeard/webserve.py b/sickbeard/webserve.py
index df77489a..5af09213 100644
--- a/sickbeard/webserve.py
+++ b/sickbeard/webserve.py
@@ -1476,7 +1476,9 @@ class ConfigProviders:
 
                 cur_name, cur_url, cur_key = curNewznabProviderStr.split('|')
                 cur_url = config.clean_url(cur_url)
+
                 newProvider = newznab.NewznabProvider(cur_name, cur_url, key=cur_key)
+
                 cur_id = newProvider.getID()
 
                 # if it already exists then update it
@@ -1489,38 +1491,11 @@ class ConfigProviders:
                         newznabProviderDict[cur_id].needs_auth = False
                     else:
                         newznabProviderDict[cur_id].needs_auth = True
-
-                    try:
-                        newznabProviderDict[cur_id].search_mode = str(kwargs[cur_id + '_search_mode']).strip()
-                    except:
-                        newznabProviderDict[cur_id].search_mode = 'eponly'
-
-                    try:
-                        newznabProviderDict[cur_id].search_fallback = config.checkbox_to_value(
-                            kwargs[cur_id + '_search_fallback'])
-                    except:
-                        newznabProviderDict[cur_id].search_fallback = 0
                 else:
-                    try:
-                        newProvider.search_mode = str(kwargs[cur_id + '_search_mode']).strip()
-                    except:
-                        newProvider.search_mode = 'eponly'
-
-                    try:
-                        newProvider.search_fallback = config.checkbox_to_value(
-                            kwargs[cur_id + '_search_fallback'])
-                    except:
-                        newProvider.search_fallback = 0
-
                     sickbeard.newznabProviderList.append(newProvider)
 
                 finishedNames.append(cur_id)
 
-            # delete anything that is missing
-            for curProvider in sickbeard.newznabProviderList:
-                if curProvider.getID() not in finishedNames:
-                    sickbeard.newznabProviderList.remove(curProvider)
-
         # delete anything that is missing
         for curProvider in sickbeard.newznabProviderList:
             if curProvider.getID() not in finishedNames:
@@ -1553,7 +1528,6 @@ class ConfigProviders:
                 finishedNames.append(curID)
 
         # delete anything that is missing
-        #logger.log(u"sickbeard.anyRssProviderList =  " + repr(sickbeard.anyRssProviderList))
         for curProvider in sickbeard.torrentRssProviderList:
             if curProvider.getID() not in finishedNames:
                 sickbeard.torrentRssProviderList.remove(curProvider)
@@ -1563,16 +1537,15 @@ class ConfigProviders:
             curProvider, curEnabled = curProviderStr.split(':')
             curEnabled = config.to_int(curEnabled)
 
+            curProvObj = [x for x in sickbeard.providers.sortedProviderList() if x.getID() == curProvider and hasattr(x, 'enabled')]
+            if curProvObj:
+                curProvObj[0].enabled = bool(curEnabled)
+
             provider_list.append(curProvider)
-
-            # dynamically set providers enabled/disabled
-            providers = sickbeard.providers.sortedProviderList()
-            for provider in providers:
-                if provider.getID() != curProvider or not hasattr(provider, 'enabled'):
-                    continue
-
-                provider.enabled = curEnabled
-                break
+            if curProvider in newznabProviderDict:
+                newznabProviderDict[curProvider].enabled = bool(curEnabled)
+            elif curProvider in torrentRssProviderDict:
+                torrentRssProviderDict[curProvider].enabled = bool(curEnabled)
 
         # dynamically load provider settings
         for curTorrentProvider in [curProvider for curProvider in sickbeard.providers.sortedProviderList() if