Breaking start_session out as a seperate function since it felt cleaner

diff --git a/src/gtk_rtmilk.py b/src/gtk_rtmilk.py
index e92c0fb..d151d1a 100644 (file)
--- a/src/gtk_rtmilk.py
+++ b/src/gtk_rtmilk.py
@@ -54,6 +54,13 @@ def get_token(username, apiKey, secret):
        return token
 
 
+def start_session(credentialsDialog):
+       # @todo Figure out storage of credentials
+       username, password = credentialsDialog.request_credentials()
+       token = get_token(username, rtmilk.RtMilkManager.API_KEY, rtmilk.RtMilkManager.SECRET)
+       return rtmilk.RtMilkManager(username, password, token)
+
+
 class GtkRtMilk(object):
 
        ID_IDX = 0
@@ -138,16 +145,11 @@ class GtkRtMilk(object):
        def name():
                return "Remember The Milk"
 
-       def start_session(self):
-               username, password = self._credentials.request_credentials()
-               token = get_token(username, rtmilk.RtMilkManager.API_KEY, rtmilk.RtMilkManager.SECRET)
-               return rtmilk.RtMilkManager(username, password, token)
-
        def enable(self):
                """
                @note UI Thread
                """
-               self._manager = self.start_session()
+               self._manager = start_session(self._credentials)
 
                self._projectsList.clear()
                self._populate_projects()