Fixed bug that removed all markets when filtering for OnlyTrade - Githubissues

sthewissen / Mynt

An Azure Functions-based crypto currency trading bot; featuring 10 exchanges, 25 indicators, custom strategy support, backtester and more

BSD 3-Clause "New" or "Revised" License

237 stars 129 forks source link

Fixed bug that removed all markets when filtering for OnlyTrade #55

Closed bogdanbujdea closed 6 years ago

bogdanbujdea commented 6 years ago

If in the OnlyTrade list you have more than one coin, then the markets list will be empty after the filter is made.

For example: OnlyTrade = ["ETC", "ETH"] markets = ["XRP", "ETC", "ETH"]

Expected behavior:

after the filter, markets should only have ETC and ETH

Actual behavior:

markets will be empty because RemoveAll will remove all coins that are not ETC, thus removing XRP and ETH from the list, then it will remove all coins that are not ETH, so it will remove ETC which will leave the list empty.