demus/openhab-addons
1
0
Fork 0
mirror of https://github.com/openhab/openhab-addons.git synced 2025-01-10 15:11:59 +01:00
Code Issues Packages Projects Releases Wiki Activity

[hue] Catch exception of 'AllGroup' does not exist (#9502)

Browse Source 
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
This commit is contained in:
Christoph Weitkamp 2020-12-25 15:44:03 +01:00
parent 4759909ff4
commit f08758aad8
No known key found for this signature in database
GPG Key ID: E60BC5062C9F0695
2 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
bundles/org.openhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/HueBridge.java
View File

@ -12,6 +12,7 @@
*/
package org.openhab.binding.hue.internal;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.UnsupportedEncodingException;
import java.lang.reflect.Type;
@ -418,8 +419,6 @@ public class HueBridge {
* @return all lights pseudo group
*/
public Group getAllGroup() {
requireAuthentication();
return new Group();
}
@ -441,14 +440,21 @@ public class HueBridge {
if (groupMap.get("0") == null) {
// Group 0 is not returned, we create it as in fact it exists
groupList.add(getGroup(new Group()));
try {
groupList.add(getGroup(getAllGroup()));
} catch (FileNotFoundException e) {
// We need a special exception handling here to further support deCONZ REST API. On deCONZ group "0" may
// not exist and the APIs will return a different HTTP status code if requesting a non existing group
// (Hue: 200, deCONZ: 404).
// see https://github.com/openhab/openhab-addons/issues/9175
logger.debug("Cannot find AllGroup with id \"0\" on Hue Bridge. Skipping it.");
}
}
for (String id : groupMap.keySet()) {
FullGroup group = groupMap.get(id);
groupMap.forEach((id, group) -> {
group.setId(id);
groupList.add(group);
}
});
return groupList;
}

2
bundles/org.openhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/handler/HueBridgeHandler.java
View File

@ -139,7 +139,7 @@ public class HueBridgeHandler extends ConfigStatusBridgeHandler implements HueCl
}
} catch (ApiException | IOException e) {
if (hueBridge != null && lastBridgeConnectionState) {
logger.debug("Connection to Hue Bridge {} lost.", hueBridge.getIPAddress());
logger.debug("Connection to Hue Bridge {} lost: {}", hueBridge.getIPAddress(), e.getMessage(), e);
lastBridgeConnectionState = false;
onConnectionLost();
}