Try next on failed download from provider

couchpotato/core/plugins/searcher/main.py

@@ -239,7 +239,7 @@ class Searcher(Plugin):
         filedata = None
         if data.get('download') and (ismethod(data.get('download')) or isfunction(data.get('download'))):
             filedata = data.get('download')(url = data.get('url'), nzb_id = data.get('id'))
-            if filedata is 'try_next':
+            if filedata == 'try_next':
                 return filedata
 
         successful = fireEvent('download', data = data, movie = movie, manual = manual, filedata = filedata, single = True)

couchpotato/core/providers/base.py

@@ -6,6 +6,7 @@ from couchpotato.environment import Env
 from urlparse import urlparse
 import re
 import time
+import traceback
 
 
 log = CPLog(__name__)
@@ -56,7 +57,12 @@ class YarrProvider(Provider):
         addEvent('nzb.feed', self.feed)
 
     def download(self, url = '', nzb_id = ''):
-        return self.urlopen(url)
+        try:
+            return self.urlopen(url, headers = {'User-Agent': Env.getIdentifier()}, show_error = False)
+        except:
+            log.error('Failed getting nzb from %s: %s', (self.getName(), traceback.format_exc()))
+
+        return 'try_next'
 
     def feed(self):
         return []

couchpotato/core/providers/nzb/newzbin/main.py

@@ -144,7 +144,8 @@ class Newzbin(NZBProvider, RSS):
             }, show_error = False)
         except Exception, e:
             log.error('Failed downloading from newzbin, check credit: %s', e)
-            return False
+
+        return 'try_next'
 
     def getFormatId(self, format):
         for id, quality in self.format_ids.iteritems():

couchpotato/core/providers/nzb/newznab/main.py

@@ -219,4 +219,5 @@ class Newznab(NZBProvider, RSS):
                     return 'try_next'
 
             log.error('Failed download from %s', (host, traceback.format_exc()))
-            raise
+
+        return 'try_next'

couchpotato/core/providers/nzb/nzbmatrix/main.py

@@ -98,9 +98,6 @@ class NZBMatrix(NZBProvider, RSS):
 
         return results
 
-    def download(self, url = '', nzb_id = ''):
-        return self.urlopen(url, headers = {'User-Agent': Env.getIdentifier()})
-
     def getApiExt(self):
         return '&username=%s&apikey=%s' % (self.conf('username'), self.conf('api_key'))
 

couchpotato/core/providers/nzb/nzbsrus/main.py

@@ -97,8 +97,5 @@ class Nzbsrus(NZBProvider, RSS):
 
         return results
 
-    def download(self, url = '', nzb_id = ''):
-        return self.urlopen(url, headers = {'User-Agent': Env.getIdentifier()})
-
     def getApiExt(self):
         return '/%s/' % (self.conf('userid'))