From 176bae2d598dac5d5d42c2d35b93d5915efd56e1 Mon Sep 17 00:00:00 2001 From: Matthias Date: Mon, 17 Sep 2018 19:57:47 +0200 Subject: [PATCH] Set default-db url in configuration, not arguments * Fixes a bug in plot_dataframe.py (#1217) * db_url is eventually overwritten here anyway. --- freqtrade/arguments.py | 1 - freqtrade/configuration.py | 5 ++++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/freqtrade/arguments.py b/freqtrade/arguments.py index 501c1784f..bb571b4ea 100644 --- a/freqtrade/arguments.py +++ b/freqtrade/arguments.py @@ -119,7 +119,6 @@ class Arguments(object): help='Override trades database URL, this is useful if dry_run is enabled' ' or in custom deployments (default: %(default)s)', dest='db_url', - default=constants.DEFAULT_DB_PROD_URL, type=str, metavar='PATH', ) diff --git a/freqtrade/configuration.py b/freqtrade/configuration.py index 3da432b1d..4e1137f33 100644 --- a/freqtrade/configuration.py +++ b/freqtrade/configuration.py @@ -110,9 +110,12 @@ class Configuration(object): '(not applicable with Backtesting and Hyperopt)' ) - if self.args.db_url != constants.DEFAULT_DB_PROD_URL: + if self.args.db_url and self.args.db_url != constants.DEFAULT_DB_PROD_URL: config.update({'db_url': self.args.db_url}) logger.info('Parameter --db-url detected ...') + else: + # Set default here + config.update({'db_url': constants.DEFAULT_DB_PROD_URL}) if config.get('dry_run', False): logger.info('Dry run is enabled')