fix(trader): Ensure consolidation of only same-currency trades

Changes include:
* unbinding _private_, and pure methods from prototype of a `Trader` object:
  * `Trader.prototype._findExchange` => `findExchange`,
  * `Trader.prototype._findTicker` => `findTicker`,
  * `Trader.prototype._findTrader` => `findTrader`,
  * `Trader.prototype._findWallet` => `findWallet`.

* Some local variables renamed to improve readability:
  * `tradeRec` => `consolidatedTrade`
  * `tickerExchangeCode` => `tickerName`,
  * `tickerExchangeConfig` => `tickerConfig`,
  * `tradeExchangeCode` => `traderName`,
  * `tradeExchangeCode` => `traderConfig`,
  * `transferExchangeCode` => `walletName`,
  * `transferExchangeConfig` => `walletConfig`.

* Since addition is commutative and order of looping doesn't matter: `while (true)` loop changed to `for..in`,
* `consolidateOther` and `otherCurencies` are used to consolidate if other currencies occur in queue.

Usually I'd prefer the for loop, because it avoids creating an anonymous function, but it's a matter of taste and they're both fine. Let's put a space between the for and the paren, and around operators: ```for (var i = 0; i < queue.length; i++) {```