From 96c233d4b9a78414b493ba628446331ccd00b9ff Mon Sep 17 00:00:00 2001 From: Fabian Affolter Date: Tue, 18 Dec 2018 15:23:53 +0100 Subject: [PATCH] Use string foratting (#19427) --- .../components/sensor/brottsplatskartan.py | 38 ++++++++++--------- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git a/homeassistant/components/sensor/brottsplatskartan.py b/homeassistant/components/sensor/brottsplatskartan.py index 7d755f773be..dc338e6b84a 100644 --- a/homeassistant/components/sensor/brottsplatskartan.py +++ b/homeassistant/components/sensor/brottsplatskartan.py @@ -1,4 +1,9 @@ -"""Sensor for Brottsplatskartan.""" +""" +Sensor platform for Brottsplatskartan information. + +For more details about this platform, please refer to the documentation at +https://home-assistant.io/components/sensor.brottsplatskartan/ +""" from collections import defaultdict from datetime import timedelta import logging @@ -17,7 +22,9 @@ REQUIREMENTS = ['brottsplatskartan==0.0.1'] _LOGGER = logging.getLogger(__name__) CONF_AREA = 'area' + DEFAULT_NAME = 'Brottsplatskartan' + SCAN_INTERVAL = timedelta(minutes=30) AREAS = [ @@ -29,37 +36,32 @@ AREAS = [ "Örebro län", "Östergötlands län" ] -PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend( - { - vol.Inclusive(CONF_LATITUDE, 'coordinates'): cv.latitude, - vol.Inclusive(CONF_LONGITUDE, 'coordinates'): cv.longitude, - vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, - vol.Optional(CONF_AREA, default=[]): - vol.All(cv.ensure_list, [vol.In(AREAS)]) - } -) +PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({ + vol.Inclusive(CONF_LATITUDE, 'coordinates'): cv.latitude, + vol.Inclusive(CONF_LONGITUDE, 'coordinates'): cv.longitude, + vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, + vol.Optional(CONF_AREA, default=[]): + vol.All(cv.ensure_list, [vol.In(AREAS)]), +}) def setup_platform(hass, config, add_entities, discovery_info=None): """Set up the Brottsplatskartan platform.""" import brottsplatskartan + + area = config.get(CONF_AREA) latitude = config.get(CONF_LATITUDE, hass.config.latitude) longitude = config.get(CONF_LONGITUDE, hass.config.longitude) name = config.get(CONF_NAME) - area = config.get(CONF_AREA) # Every Home Assistant instance should have their own unique # app parameter: https://brottsplatskartan.se/sida/api - app = 'ha-' + str(uuid.getnode()) + app = 'ha-{}'.format(uuid.getnode()) bpk = brottsplatskartan.BrottsplatsKartan( - app=app, area=area, - latitude=latitude, longitude=longitude - ) + app=app, area=area, latitude=latitude, longitude=longitude) - add_entities( - [BrottsplatskartanSensor(bpk, name)], True - ) + add_entities([BrottsplatskartanSensor(bpk, name)], True) class BrottsplatskartanSensor(Entity):