ED/EDMarketConnector
1
0
Fork 0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-14 08:17:13 +03:00
Code Issues Projects Releases Wiki Activity

Simplified if ladder

Browse Source 
Removed a large if ladder with a simpler construct that does the same
thing
This commit is contained in:
A_D 2020-07-07 16:59:03 +02:00
parent 31e0758ee4
commit 9363b1457d
No known key found for this signature in database
GPG Key ID: 4BE9EB7DF45076C4
1 changed files with 6 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
companion.py
View File

@ -236,20 +236,10 @@ class Auth(object):
if not data.get('code'):
print('Auth\tNegative response {!r}'.format(payload))
# TODO(A_D): there should be a cleaner way to do this rather than raising in every if
# Additionally, this is basically the same code as seen below, helper method perhaps?
if data.get('error_description'):
raise CredentialsError('Error: {!r}'.format(data['error_description'][0]))
elif data.get('error'):
raise CredentialsError('Error: {!r}'.format(data['error'][0]))
elif data.get('message'):
raise CredentialsError('Error: {!r}'.format(data['message'][0]))
else:
raise CredentialsError()
error = next(
(data[k] for k in ('error_description', 'error', 'message') if k in data), ('<unknown error>',)
)
raise CredentialsError('Error: {!r}'.format(error)[0])
try:
r = None
@ -285,17 +275,8 @@ class Auth(object):
print('Auth\tCan\'t get token for {}'.format(self.cmdr))
self.dump(r)
if data.get('error_description'):
raise CredentialsError('Error: {!r}'.format(data['error_description']))
elif data.get('error'):
raise CredentialsError('Error: {!r}'.format(data['error']))
elif data.get('message'):
raise CredentialsError('Error: {!r}'.format(data['message']))
else:
raise CredentialsError()
error = next((data[k] for k in ('error_description', 'error', 'message') if k in data), ('<unknown error>',))
raise CredentialsError('Error: {!r}'.format(error)[0])
@staticmethod
def invalidate(cmdr):