Jessecar96/SteamBot — pull request #102 by cwhelchel (2012-11-29 01:40:14+00:00)
Title: Updates for the inventory fetching.

This pull request includes a new ``TradeManager`` class that is responsible for creating and starting trades. No longer will the ``Bot`` create ``Trade`` objects. It will use them, just not call the epic-ly long ``Trade`` constructor.

This makes the SteamTrade library more library-like by controlling creation of the most important part of the library, the ``Trade`` objects.

The thread that called ``Trade.Poll`` has also been moved into the ``Trade`` class because it doesn't make sense to clutter up the ``Bot`` class with it.

As far as the inventory fetching issues #98 and #99, the should be addressed so that it won't crash the bot. Instead, the bot will message the other user that it couldn't get their inventory or that their inventory was private.

> I don't think there's a way to get trading to work without fetching the other trade partner and the bot's inventory.

Oh and this also adds a new exception type: ``InventoryFetchException`` (#89)