jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-23 13:17:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

bugfix for actiontec device tracker

Browse Source
This commit is contained in:
Nolan Gilley 2015-10-28 17:20:15 -04:00
parent 2eb65c8eb8
commit 3b37a7b737
1 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
homeassistant/components/device_tracker/actiontec.py
View File

@ -30,7 +30,9 @@ _LOGGER = logging.getLogger(__name__)
_LEASES_REGEX = re.compile( _LEASES_REGEX = re.compile(
r'(?P<ip>([0-9]{1,3}[\.]){3}[0-9]{1,3})' + r'(?P<ip>([0-9]{1,3}[\.]){3}[0-9]{1,3})' +
r'\smac:\s(?P<mac>([0-9a-f]{2}[:-]){5}([0-9a-f]{2}))') r'\smac:\s(?P<mac>([0-9a-f]{2}[:-]){5}([0-9a-f]{2}))' +
r'\svalid\sfor:\s(?P<timevalid>(-?\d+))' +
r'\ssec')
# pylint: disable=unused-argument # pylint: disable=unused-argument
@ -40,9 +42,7 @@ def get_scanner(hass, config):
{DOMAIN: [CONF_HOST, CONF_USERNAME, CONF_PASSWORD]}, {DOMAIN: [CONF_HOST, CONF_USERNAME, CONF_PASSWORD]},
_LOGGER): _LOGGER):
return None return None
scanner = ActiontecDeviceScanner(config[DOMAIN]) scanner = ActiontecDeviceScanner(config[DOMAIN])
return scanner if scanner.success_init else None return scanner if scanner.success_init else None
Device = namedtuple("Device", ["mac", "ip", "last_update"]) Device = namedtuple("Device", ["mac", "ip", "last_update"])
@ -60,11 +60,8 @@ class ActiontecDeviceScanner(object):
self.password = config[CONF_PASSWORD] self.password = config[CONF_PASSWORD]
minutes = convert(config.get(CONF_HOME_INTERVAL), int, 0) minutes = convert(config.get(CONF_HOME_INTERVAL), int, 0)
self.home_interval = timedelta(minutes=minutes) self.home_interval = timedelta(minutes=minutes)
self.lock = threading.Lock() self.lock = threading.Lock()
self.last_results = [] self.last_results = []
# Test the router is accessible # Test the router is accessible
data = self.get_actiontec_data() data = self.get_actiontec_data()
self.success_init = data is not None self.success_init = data is not None
@ -109,7 +106,6 @@ class ActiontecDeviceScanner(object):
exclude_targets.add(host) exclude_targets.add(host)
if len(exclude_targets) > 0: if len(exclude_targets) > 0:
exclude_target_list = [t.ip for t in exclude_targets] exclude_target_list = [t.ip for t in exclude_targets]
actiontec_data = self.get_actiontec_data() actiontec_data = self.get_actiontec_data()
if not actiontec_data: if not actiontec_data:
return False return False
@ -118,8 +114,9 @@ class ActiontecDeviceScanner(object):
if client in actiontec_data: if client in actiontec_data:
actiontec_data.pop(client) actiontec_data.pop(client)
for name, data in actiontec_data.items(): for name, data in actiontec_data.items():
device = Device(data['mac'], name, now) if data['timevalid'] > 0:
self.last_results.append(device) device = Device(data['mac'], name, now)
self.last_results.append(device)
self.last_results.extend(exclude_targets) self.last_results.extend(exclude_targets)
_LOGGER.info("actiontec scan successful") _LOGGER.info("actiontec scan successful")
return True return True
@ -153,6 +150,7 @@ class ActiontecDeviceScanner(object):
if match is not None: if match is not None:
devices[match.group('ip')] = { devices[match.group('ip')] = {
'ip': match.group('ip'), 'ip': match.group('ip'),
'mac': match.group('mac').upper() 'mac': match.group('mac').upper(),
'timevalid': int(match.group('timevalid'))
} }
return devices return devices