From 748d7f4ecb63d9a57be497bcfe2768eb0ac5e0ca Mon Sep 17 00:00:00 2001 From: Fabian Affolter Date: Thu, 1 Sep 2016 21:57:47 +0200 Subject: [PATCH] Bitcoin sensor use warning instead of error (#3103) --- homeassistant/components/sensor/bitcoin.py | 27 ++++++++++++---------- 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/homeassistant/components/sensor/bitcoin.py b/homeassistant/components/sensor/bitcoin.py index 0d859314bb4..51b5f9bba3b 100644 --- a/homeassistant/components/sensor/bitcoin.py +++ b/homeassistant/components/sensor/bitcoin.py @@ -17,6 +17,16 @@ from homeassistant.util import Throttle REQUIREMENTS = ['blockchain==1.3.3'] +_LOGGER = logging.getLogger(__name__) + +CONF_CURRENCY = 'currency' + +DEFAULT_CURRENCY = 'USD' + +ICON = 'mdi:currency-btc' + +MIN_TIME_BETWEEN_UPDATES = timedelta(minutes=5) + OPTION_TYPES = { 'exchangerate': ['Exchange rate (1 BTC)', None], 'trade_volume_btc': ['Trade volume', 'BTC'], @@ -41,20 +51,12 @@ OPTION_TYPES = { 'market_price_usd': ['Market price', 'USD'] } -ICON = 'mdi:currency-btc' -CONF_CURRENCY = 'currency' - PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({ vol.Required(CONF_DISPLAY_OPTIONS, default=[]): - [vol.In(OPTION_TYPES)], - vol.Optional(CONF_CURRENCY, default='USD'): cv.string, + vol.All(cv.ensure_list, [vol.In(OPTION_TYPES)]), + vol.Optional(CONF_CURRENCY, default=DEFAULT_CURRENCY): cv.string, }) -_LOGGER = logging.getLogger(__name__) - -# Return cached results if last scan was less then this time ago. -MIN_TIME_BETWEEN_UPDATES = timedelta(minutes=5) - def setup_platform(hass, config, add_devices, discovery_info=None): """Setup the Bitcoin sensors.""" @@ -63,8 +65,9 @@ def setup_platform(hass, config, add_devices, discovery_info=None): currency = config.get(CONF_CURRENCY) if currency not in exchangerates.get_ticker(): - _LOGGER.error('Currency "%s" is not available. Using "USD"', currency) - currency = 'USD' + _LOGGER.warning('Currency "%s" is not available. Using "USD"', + currency) + currency = DEFAULT_CURRENCY data = BitcoinData() dev = []