From c67c20f75205ad6da5c120a3736847a2eb9e9c1e Mon Sep 17 00:00:00 2001 From: Mike Christianson Date: Mon, 17 Jul 2017 10:45:42 -0700 Subject: [PATCH] fix for a bug introduced with media support in #8282 (#8513) data may be None if twitter data property unconfigured: File "/opt/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/notify/twitter.py", line 63, in send_message media = data.get(ATTR_MEDIA) --- homeassistant/components/notify/twitter.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/homeassistant/components/notify/twitter.py b/homeassistant/components/notify/twitter.py index 6d74f86132a..43efd2c2732 100644 --- a/homeassistant/components/notify/twitter.py +++ b/homeassistant/components/notify/twitter.py @@ -60,10 +60,13 @@ class TwitterNotificationService(BaseNotificationService): def send_message(self, message="", **kwargs): """Tweet a message, optionally with media.""" data = kwargs.get(ATTR_DATA) - media = data.get(ATTR_MEDIA) - if not self.hass.config.is_allowed_path(media): - _LOGGER.warning("'%s' is not in a whitelisted area.", media) - return + + media = None + if data: + media = data.get(ATTR_MEDIA) + if not self.hass.config.is_allowed_path(media): + _LOGGER.warning("'%s' is not in a whitelisted area.", media) + return media_id = self.upload_media(media)