public/openhab-addons
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[sonos] Avoid empty thing (type) ID in discovery (#12626)

Browse Source 
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
This commit is contained in:
lolodomo 2022-04-18 19:36:56 +02:00 committed by GitHub
parent e42202d2e5
commit 6b56b40fe1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
bundles/org.openhab.binding.sonos/src/main/java/org/openhab/binding/sonos/internal/discovery/ZonePlayerDiscoveryParticipant.java
View File

@ -82,7 +82,8 @@ public class ZonePlayerDiscoveryParticipant implements UpnpDiscoveryParticipant
if (device.getDetails().getManufacturerDetails().getManufacturer().toUpperCase().contains("SONOS")) {
String id = SonosXMLParser
.buildThingTypeIdFromModelName(device.getDetails().getModelDetails().getModelName());
if (!"Sub".equalsIgnoreCase(id)) {
String udn = device.getIdentity().getUdn().getIdentifierString();
if (!id.isEmpty() && !"Sub".equalsIgnoreCase(id) && !udn.isEmpty()) {
ThingTypeUID thingTypeUID = new ThingTypeUID(SonosBindingConstants.BINDING_ID, id);
if (!SonosBindingConstants.SUPPORTED_KNOWN_THING_TYPES_UIDS.contains(thingTypeUID)) {
// Try with the model name all in uppercase
@ -96,9 +97,8 @@ public class ZonePlayerDiscoveryParticipant implements UpnpDiscoveryParticipant
}
}
logger.debug("Discovered a Sonos '{}' thing with UDN '{}'", thingTypeUID,
device.getIdentity().getUdn().getIdentifierString());
return new ThingUID(thingTypeUID, device.getIdentity().getUdn().getIdentifierString());
logger.debug("Discovered a Sonos '{}' thing with UDN '{}'", thingTypeUID, udn);
return new ThingUID(thingTypeUID, udn);
}
}
}