Skip to content

Commit

Permalink
[homematic] Checking of full datapoint name is necessary (openhab#8331)
Browse files Browse the repository at this point in the history
* Test of address only was not sufficient.
* Another exception for data point added. According to:
openhab#5048 (comment)

Additional fix for openhab#5048

Signed-off-by: Martin Herbst <[email protected]>
  • Loading branch information
MHerbst authored and CSchlipp committed Sep 12, 2020
1 parent 14bb130 commit a055c99
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,10 @@ public Void parse(Object[] message) throws IOException {
// suppress warning for this datapoint due wrong CCU metadata
String deviceType = channel.getDevice().getType();
boolean isHmSenMdirNextTrans = dpInfo.getName().equals("NEXT_TRANSMISSION")
&& (deviceType.startsWith("HM-Sen-MDIR-O") || deviceType.startsWith("HM-Sen-MDIR-WM55"));
&& (deviceType.startsWith("HM-Sen-MDIR-O") || deviceType.startsWith("HM-Sen-MDIR-WM55")
|| deviceType.startsWith("HM-Sec-MDIR-2"));
if (!isHmSenMdirNextTrans) {
if (dpInfo.getAddress().contains(":M_")
if (dpInfo.getParamsetType() == HmParamsetType.MASTER
&& channel.getDevice().getHmInterface() == HmInterface.HMIP) {
// These data points can't currently be recognized and therefore can't be created
logger.debug("Can't set value for channel configuration datapoint '{}'", dpInfo);
Expand Down

0 comments on commit a055c99

Please sign in to comment.