From 1f1770ad5a21e5c994fd46041b183fab6db2cc34 Mon Sep 17 00:00:00 2001
From: misagh <misagh.shakeri@gmail.com>
Date: Mon, 26 Nov 2018 18:46:59 +0100
Subject: [PATCH] migration script and and error handling on stop loss order

---
 freqtrade/exchange/__init__.py      | 19 ++++++++++++++++++-
 freqtrade/persistence.py            |  2 +-
 freqtrade/tests/test_persistence.py |  1 +
 3 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/freqtrade/exchange/__init__.py b/freqtrade/exchange/__init__.py
index a311fd666..38e2fa317 100644
--- a/freqtrade/exchange/__init__.py
+++ b/freqtrade/exchange/__init__.py
@@ -372,9 +372,26 @@ class Exchange(object):
             }
             return self._dry_run_open_orders[order_id]
 
-        return self._api.create_order(pair, 'stop_loss', 'sell',
+        try:
+            return self._api.create_order(pair, 'stop_loss', 'sell',
                                       amount, rate, {'stopPrice': stop_price})
 
+        except ccxt.InsufficientFunds as e:
+            raise DependencyException(
+                f'Insufficient funds to place stoploss limit order on market {pair}.'
+                f'Tried to put a stoploss amount {amount} at rate {rate} (total {rate*amount}).'
+                f'Message: {e}')
+        except ccxt.InvalidOrder as e:
+            raise DependencyException(
+                f'Could not place stoploss limit order on market {pair}.'
+                f'Tried to place stoploss amount {amount} at rate {rate} (total {rate*amount}).'
+                f'Message: {e}')
+        except (ccxt.NetworkError, ccxt.ExchangeError) as e:
+            raise TemporaryError(
+                f'Could not place stoploss limit order due to {e.__class__.__name__}. Message: {e}')
+        except ccxt.BaseError as e:
+            raise OperationalException(e)
+
     @retrier
     def get_balance(self, currency: str) -> float:
         if self._conf['dry_run']:
diff --git a/freqtrade/persistence.py b/freqtrade/persistence.py
index 364af06ce..26b0d9d93 100644
--- a/freqtrade/persistence.py
+++ b/freqtrade/persistence.py
@@ -82,7 +82,7 @@ def check_migrate(engine) -> None:
         logger.debug(f'trying {table_back_name}')
 
     # Check for latest column
-    if not has_column(cols, 'ticker_interval'):
+    if not has_column(cols, 'stoploss_order_id'):
         logger.info(f'Running database migration - backup available as {table_back_name}')
 
         fee_open = get_column_def(cols, 'fee_open', 'fee')
diff --git a/freqtrade/tests/test_persistence.py b/freqtrade/tests/test_persistence.py
index 5e0647dff..cdfdef6e6 100644
--- a/freqtrade/tests/test_persistence.py
+++ b/freqtrade/tests/test_persistence.py
@@ -426,6 +426,7 @@ def test_migrate_new(mocker, default_conf, fee, caplog):
                                 max_rate FLOAT,
                                 sell_reason VARCHAR,
                                 strategy VARCHAR,
+                                ticker_interval INTEGER,
                                 PRIMARY KEY (id),
                                 CHECK (is_open IN (0, 1))
                                 );"""