From cbe27d8f1a1cd1b8b67204fdfd5fde599ac94323 Mon Sep 17 00:00:00 2001
From: Gert <gert.joos@gmail.com>
Date: Mon, 22 Feb 2016 11:44:00 +0100
Subject: [PATCH] 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.
---
homeassistant/components/sensor/bitcoin.py | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/homeassistant/components/sensor/bitcoin.py b/homeassistant/components/sensor/bitcoin.py
index fe17ff574b1..62d1622aff4 100644
--- a/homeassistant/components/sensor/bitcoin.py
+++ b/homeassistant/components/sensor/bitcoin.py
@@ -59,12 +59,14 @@ def setup_platform(hass, config, add_devices, discovery_info=None):
_LOGGER.error('Currency "%s" is not available. Using "USD".', currency)
currency = 'USD'
- wallet = Wallet(wallet_id, password)
-
- try:
- wallet.get_balance()
- except exceptions.APIException as error:
- _LOGGER.error(error)
+ if wallet_id is not None and password is not None:
+ wallet = Wallet(wallet_id, password)
+ try:
+ wallet.get_balance()
+ except exceptions.APIException as error:
+ _LOGGER.error(error)
+ wallet = None
+ else:
wallet = None
data = BitcoinData()
--
GitLab