Home-Assistant
/
core
mirror of https://github.com/home-assistant/core.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't set wallet if not setup in configuration 

Because of problems with the Wallet part of python blockchain library (see #1242 ) , the entire Bitcoin module isn't working currently.
This change does not fix those problems but at least makes the sensor work again for people who don't need Wallet-related functionality.

It also just seems better practice to not set a wallet and call "wallet.get_balance()" when not wallet is set in configuration.
pull/1368/head
Gert 2016-02-22 11:44:00 +01:00
parent 680f450278
commit cbe27d8f1a
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
homeassistant/components/sensor/bitcoin.py
View File

@ -59,12 +59,14 @@ def setup_platform(hass, config, add_devices, discovery_info=None):
_LOGGER.error('Currency "%s" is not available. Using "USD".', currency) _LOGGER.error('Currency "%s" is not available. Using "USD".', currency)
currency = 'USD' currency = 'USD'
wallet = Wallet(wallet_id, password) if wallet_id is not None and password is not None:
wallet = Wallet(wallet_id, password)
try: try:
wallet.get_balance() wallet.get_balance()
except exceptions.APIException as error: except exceptions.APIException as error:
_LOGGER.error(error) _LOGGER.error(error)
wallet = None
else:
wallet = None wallet = None
data = BitcoinData() data = BitcoinData()