From 8d1afb7e4ebb99c3315e118e4deeb445baff1bf5 Mon Sep 17 00:00:00 2001
From: Scott <scott.grant@live.com.au>
Date: Fri, 12 Aug 2016 22:44:25 +1000
Subject: [PATCH] Fixes issue with first and last index of enabled currency
 array destroying config

---
 web/app/views/settings/settings.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/web/app/views/settings/settings.js b/web/app/views/settings/settings.js
index b6b342f1..a8dca613 100644
--- a/web/app/views/settings/settings.js
+++ b/web/app/views/settings/settings.js
@@ -33,8 +33,12 @@ $scope.toggleCurrencyToEnabledCurrencies = function(currency, exchange) {
   for(var i=0; i<$scope.config.Exchanges.length;i++) {
     if($scope.config.Exchanges[i].Name == exchange.Name) {
       if(exchange.EnabledPairsSplit.indexOf(currency) > -1) {
-        $scope.config.Exchanges[i].EnabledPairsSplit.splice(exchange.EnabledPairsSplit.indexOf(currency),1);
-        $scope.config.Exchanges[i].EnabledPairs  = $scope.config.Exchanges[i].EnabledPairs.replace(","+ currency,"");
+        $scope.config.Exchanges[i].EnabledPairsSplit.splice($scope.config.Exchanges[i].EnabledPairsSplit.indexOf(currency),1);
+        //I feel there's a better way to do this, but for right now, whatever
+        $scope.config.Exchanges[i].EnabledPairs  = $scope.config.Exchanges[i].EnabledPairs.replace(currency,"");
+        $scope.config.Exchanges[i].EnabledPairs  = $scope.config.Exchanges[i].EnabledPairs.replace(",,","");
+        $scope.config.Exchanges[i].EnabledPairs  = $scope.config.Exchanges[i].EnabledPairs.replace(/,\s*$/, "");
+        
       } else {
         $scope.config.Exchanges[i].EnabledPairsSplit.push(currency);
         $scope.config.Exchanges[i].EnabledPairs  = $scope.config.Exchanges[i].EnabledPairs + "," + currency;