Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'list' object has no attribute 'keys' #15

Closed
seeebek opened this issue Jul 8, 2015 · 4 comments
Closed

'list' object has no attribute 'keys' #15

seeebek opened this issue Jul 8, 2015 · 4 comments
Labels

Comments

@seeebek
Copy link

seeebek commented Jul 8, 2015

After the last update there is this error:
error

Traceback (most recent call last):
  File "C:\Users\Sebastian\Downloads\EDMarketConnector-master (1)\EDMarketConnector-master\EDMarketConnector.py", line 240, in getandsend
    eddn.export(data)
  File "C:\Users\Sebastian\Downloads\EDMarketConnector-master (1)\EDMarketConnector-master\eddn.py", line 100, in export
    'ships'       : [ship_map.get(ship,ship) for ship in (data['lastStarport']['ships'].get('shipyard_list') or {}).keys() + (data['lastStarport']['ships'].get('unavailable_list') or {}).keys()],
AttributeError: 'list' object has no attribute 'keys'
@Marginal
Copy link
Contributor

Marginal commented Jul 8, 2015

Dealing with the Companion API is like playing Twenty Questions!
I suspect the problem is with 'unavailable_list' - I've only ever seen it empty so I had to guess at what it might look like.

Please mail me the .json file that was produced when you ran from source in order to obtain that traceback.

@seeebek
Copy link
Author

seeebek commented Jul 8, 2015

You have mail ;-)
I think the error happens on stations without shipyards. I'm not fully sure.

@Marginal Marginal added the bug label Jul 8, 2015
@Marginal
Copy link
Contributor

Marginal commented Jul 8, 2015

Thanks. Fixed in Release 1.61.

@Marginal
Copy link
Contributor

Marginal commented Jul 8, 2015

I think the error happens on stations without shipyards.

I think this is a separate issue. I've raised Issue #16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants