Syncing Trade Init + minor log changes.

GivingUserHandler now intelligently waits for both bots to have the
trade initialized before adding items (Should help with some 'add item'
failures). Also tweaked the way errors are handled, the log will be much
cleaner, and fixed some situations where `OnTradeClose` was being called
twice, causing issues.