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

The API doesn't support empty inventories. #4

Open
KowalskiThomas opened this issue Aug 7, 2016 · 1 comment
Open

The API doesn't support empty inventories. #4

KowalskiThomas opened this issue Aug 7, 2016 · 1 comment

Comments

@KowalskiThomas
Copy link

The title says it all.

If I remember well I've already faced twice problems related to the fact you're using, for example

    for x in rgInventory.keys():

Without thinking about the fact rgInventory could be [].

That would be great if you could fix that (since I don't know every where it can happen in the code).

@DrBomb
Copy link
Owner

DrBomb commented Aug 7, 2016

Inventories are going to be recoded so I won't be fixing this for now. But if you look at the code here: https://github.com/DrBomb/python-steamcommunity/blob/master/SteamInventory.py#L48
You will see that if a inventory is empty the respective dictionary entry should be an empty dictionary

@DrBomb DrBomb added this to the Recode Inventories milestone Aug 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants