From ce2e80339c67f3a9ce6bb230f8345f13da660236 Mon Sep 17 00:00:00 2001
From: Chris Colohan <github@colohan.com>
Date: Wed, 2 Oct 2019 00:50:45 -0700
Subject: [PATCH] Add Vera last user and low battery attributes (#27043)

* Add in attributes to track when a user unlocks the lock with a PIN, and when the battery runs low.

* Vera attributes for who entered PIN on lock, and low battery warning.

* Changed last_user_id to use changed_by interface.

* Bump pyvera version to 0.3.6; remove guard code for earlier pyvera versions.

* Bump pyvera version to 0.3.6
---
 homeassistant/components/vera/lock.py       | 32 +++++++++++++++++++++
 homeassistant/components/vera/manifest.json |  2 +-
 requirements_all.txt                        |  2 +-
 3 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/homeassistant/components/vera/lock.py b/homeassistant/components/vera/lock.py
index cf2d6d25c4a..23b62bb0331 100644
--- a/homeassistant/components/vera/lock.py
+++ b/homeassistant/components/vera/lock.py
@@ -8,6 +8,9 @@ from . import VERA_CONTROLLER, VERA_DEVICES, VeraDevice
 
 _LOGGER = logging.getLogger(__name__)
 
+ATTR_LAST_USER_NAME = "changed_by_name"
+ATTR_LOW_BATTERY = "low_battery"
+
 
 def setup_platform(hass, config, add_entities, discovery_info=None):
     """Find and return Vera locks."""
@@ -44,6 +47,35 @@ class VeraLock(VeraDevice, LockDevice):
         """Return true if device is on."""
         return self._state == STATE_LOCKED
 
+    @property
+    def device_state_attributes(self):
+        """Who unlocked the lock and did a low battery alert fire.
+
+        Reports on the previous poll cycle.
+        changed_by_name is a string like 'Bob'.
+        low_battery is 1 if an alert fired, 0 otherwise.
+        """
+        data = super().device_state_attributes
+
+        last_user = self.vera_device.get_last_user_alert()
+        if last_user is not None:
+            data[ATTR_LAST_USER_NAME] = last_user[1]
+
+        data[ATTR_LOW_BATTERY] = self.vera_device.get_low_battery_alert()
+        return data
+
+    @property
+    def changed_by(self):
+        """Who unlocked the lock.
+
+        Reports on the previous poll cycle.
+        changed_by is an integer user ID.
+        """
+        last_user = self.vera_device.get_last_user_alert()
+        if last_user is not None:
+            return last_user[0]
+        return None
+
     def update(self):
         """Update state by the Vera device callback."""
         self._state = (
diff --git a/homeassistant/components/vera/manifest.json b/homeassistant/components/vera/manifest.json
index 8365ca1a765..b2f1581e76f 100644
--- a/homeassistant/components/vera/manifest.json
+++ b/homeassistant/components/vera/manifest.json
@@ -3,7 +3,7 @@
   "name": "Vera",
   "documentation": "https://www.home-assistant.io/components/vera",
   "requirements": [
-    "pyvera==0.3.4"
+    "pyvera==0.3.6"
   ],
   "dependencies": [],
   "codeowners": []
diff --git a/requirements_all.txt b/requirements_all.txt
index 385f2d0e1a2..245f91dc9c6 100644
--- a/requirements_all.txt
+++ b/requirements_all.txt
@@ -1625,7 +1625,7 @@ pyuptimerobot==0.0.5
 # pyuserinput==0.1.11
 
 # homeassistant.components.vera
-pyvera==0.3.4
+pyvera==0.3.6
 
 # homeassistant.components.vesync
 pyvesync==1.1.0