Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

Quadriga Quick Bugfix #1948

Merged
merged 5 commits into from
Feb 21, 2018
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Quick bugfix, adding toLowerCase
  • Loading branch information
cmroche committed Feb 19, 2018
commit 850ea953e915ef6172776791ed2f4dedf069d823
4 changes: 2 additions & 2 deletions exchanges/quadriga.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ Trader.prototype.getPortfolio = function(callback) {
if (data && data.error) return this.retry(this.getPortfolio, 'unable to get balance', args, data.error);
if (err) return this.retry(this.getPortfolio, 'unable to get balance', args, err);

var assetAmount = parseFloat( data[this.asset + '_available'] );
var currencyAmount = parseFloat( data[this.currency + '_available'] );
var assetAmount = parseFloat( data[this.asset.toLowerCase() + '_available'] );
var currencyAmount = parseFloat( data[this.currency.toLowerCase() + '_available'] );

if(!_.isNumber(assetAmount) || _.isNaN(assetAmount)) {
log.error(`Quadriga did not return balance for ${this.asset.toLowerCase()}, assuming 0.`);
Expand Down