Use existing Trakt auth settings and other cleanup

couchpotato/core/notifications/base.py

@@ -2,13 +2,15 @@ from couchpotato.api import addApiView
 from couchpotato.core.event import addEvent
 from couchpotato.core.helpers.request import jsonified
 from couchpotato.core.logger import CPLog
-from couchpotato.core.plugins.base import Plugin
+from couchpotato.core.providers.base import Provider
 from couchpotato.environment import Env
 
 log = CPLog(__name__)
 
 
-class Notification(Plugin):
+class Notification(Provider):
+
+    type = 'notification'
 
     default_title = Env.get('appname')
     test_message = 'ZOMG Lazors Pewpewpew!'

couchpotato/core/notifications/trakt/__init__.py

@@ -8,9 +8,10 @@ config = [{
     'groups': [
         {
             'tab': 'notifications',
-            'name': 'trakt_notification',
+            'list': 'notification_providers',
+            'name': 'trakt',
             'label': 'Trakt',
-            'description': 'add movies to your collection once downloaded',
+            'description': 'add movies to your collection once downloaded. Fill in your username and password in the <a href="../automation/">Automation Trakt settings</a>',
             'options': [
                 {
                     'name': 'notification_enabled',
@@ -23,20 +24,6 @@ config = [{
                     'default': False,
                     'type': 'bool',
                 },
-                {
-                    'name': 'automation_api_key',
-                    'label': 'Apikey',
-                },
-                {
-                    'name': 'automation_username',
-                    'label': 'Username',
-                },
-                {
-                    'name': 'automation_password',
-                    'label': 'Password',
-                    'type': 'password',
-                    'description': 'Required even if your account is unprotected.',
-                },
             ],
         }
     ],

couchpotato/core/notifications/trakt/main.py

@@ -1,14 +1,12 @@
 from couchpotato.core.logger import CPLog
 from couchpotato.core.notifications.base import Notification
-import json
-import urllib2
 
 log = CPLog(__name__)
 
 class Trakt(Notification):
 
     urls = {
-        'base': 'http://api.trakt.tv/',
+        'base': 'http://api.trakt.tv/%s',
         'library': 'movie/library/%s',
         'unwatchlist': 'movie/unwatchlist/%s',
     }
@@ -17,17 +15,17 @@ class Trakt(Notification):
 
     def notify(self, message = '', data = {}, listener = None):
         if self.isDisabled(): return
-        if not data: return
+
         post_data = {
             'username': self.conf('automation_username'),
             'password' : self.conf('automation_password'),
-            'movies': [ {
+            'movies': [{
                 'imdb_id': data['library']['identifier'],
                 'title': data['library']['titles'][0]['title'],
                 'year': data['library']['year']
-                } ]
+            }] if data else []
-            }
+        }
-        result = False
+
         result = self.call((self.urls['library'] % self.conf('automation_api_key')), post_data)
         if self.conf('remove_watchlist_enabled'):
             result = result and self.call((self.urls['unwatchlist'] % self.conf('automation_api_key')), post_data)
@@ -35,14 +33,15 @@ class Trakt(Notification):
         return result
 
     def call(self, method_url, post_data):
-        log.info('opening url: ' + self.urls['base'] + method_url + ', post: ' + str(post_data))
+
         try:
-            response = urllib2.urlopen(self.urls['base'] + method_url, json.dumps(post_data))
+            response = self.getJsonData(self.urls['base'] % method_url, params = post_data, cache_timeout = 1)
             if response:
-                if json.load(response).get('status') == "success":
+                if response.get('status') == "success":
                     log.info('Successfully called Trakt')
                     return True
         except:
             pass
+
         log.error('Failed to call trakt, check your login.')
         return False