Skip to content
This repository has been archived by the owner on Jul 24, 2018. It is now read-only.

Add get_full_oauth_token. #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 17 additions & 17 deletions moves/_moves.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import json
import urllib
import requests
import types
Expand Down Expand Up @@ -32,11 +31,6 @@ def __init__(self, client_id=None, client_secret=None,
self.use_app = use_app
self._last_headers = None

def parse_response(self, response):
"""Parse JSON API responses."""

return json.loads(response.text)

def build_oauth_url(self, redirect_uri=None, scope="activity location"):
params = {
'client_id': self.client_id,
Expand All @@ -53,8 +47,10 @@ def build_oauth_url(self, redirect_uri=None, scope="activity location"):
encoded = urllib.urlencode(params).replace('+', '%20')
return "%s?%s" % (self.auth_url, encoded)

def get_oauth_token(self, code, **kwargs):

def get_full_oauth_token(self, code, **kwargs):
'''
Get full token info. token, expires_in...
'''
params = {
'client_id': self.client_id,
'client_secret': self.client_secret,
Expand All @@ -65,7 +61,15 @@ def get_oauth_token(self, code, **kwargs):
if 'redirect_uri' in kwargs:
params['redirect_uri'] = kwargs['redirect_uri']
response = requests.post(self.token_url, params=params)
response = json.loads(response.content)

return response.json()


def get_oauth_token(self, code, **kwargs):
'''
Get access token
'''
response = self.get_full_oauth_token(code, **kwargs)
try:
return response['access_token']
except:
Expand All @@ -79,7 +83,7 @@ def tokeninfo(self):
}

response = requests.get(self.tokeninfo_url, params=params)
response = json.loads(response.content)
response = response.json()
try:
return response
except:
Expand Down Expand Up @@ -124,12 +128,10 @@ def api(self, path, method='GET', **kwargs):
return resp

def get(self, path, **params):
return self.parse_response(
self.api(path, 'GET', params=params))
return self.api(path, 'GET', params=params).json()

def post(self, path, **data):
return self.parse_response(
self.api(path, 'POST', data=data))
return self.api(path, 'POST', data=data).json()

def set_first_date(self):
if not self.first_date:
Expand All @@ -149,9 +151,7 @@ def closure(*path, **params):
'Accesses the /%s API endpoints.'
path = list(path)
path.insert(0, base_path)
return self.parse_response(
self.api('/'.join(path), 'GET', params=params)
)
return self.api('/'.join(path), 'GET', params=params).json()

# Clone a new method with the correct name and doc string.
retval = types.FunctionType(
Expand Down