mtan93/SmartThingsPublic
1
0
Fork 0
mirror of https://github.com/mtan93/SmartThingsPublic.git synced 2026-03-21 21:03:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

DVCSMP-2425 - LIFX Exceptions

Browse Source 
Adding comment
This commit is contained in:
juano2310
2017-02-10 19:36:39 -05:00
parent 1611fd0144
commit bcc680ee5d
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
smartapps/smartthings/lifx-connect.src/lifx-connect.groovy
View File

@@ -346,7 +346,7 @@ def devicesList(selector = '') {
if (resp.status == 200) { if (resp.status == 200) {
return resp.data return resp.data
} else { } else {
log.error("Non-200 from device list call. ${resp.status} ${resp.data}") log.debug("No response from device list call. ${resp.status} ${resp.data}")
return [] return []
} }
} }
@@ -418,9 +418,15 @@ def updateDevices() {
} }
getChildDevices().findAll { !selectors.contains("${it.deviceNetworkId}") }.each { getChildDevices().findAll { !selectors.contains("${it.deviceNetworkId}") }.each {
log.info("Deleting ${it.deviceNetworkId}") log.info("Deleting ${it.deviceNetworkId}")
state.devices[it.deviceNetworkId] = null if (state.devices[it.deviceNetworkId])
deleteChildDevice(it.deviceNetworkId) state.devices[it.deviceNetworkId] = null
// The reason the implementation is trying to delete this bulb is because it is not longer connected to the LIFX location.
// Adding "try" will prevent this exception from happening.
// Ideally device health would show to the user that the device is not longer accessible so that the user can either force delete it or remove it from the SmartApp.
try {
deleteChildDevice(it.deviceNetworkId)
} catch (Exception e) {
log.debug("Can't remove this device because it's being used by an SmartApp")
}
} }
} }