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

[mqtt] recognize -NaN as UNDEF as well (#16245)

Browse Source 
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
This commit is contained in:
Cody Cutrer 2024-01-08 09:15:02 -07:00 committed by Ciprian Pascu
parent 8e34fa4ffd
commit 70137ce58a
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bundles/org.openhab.binding.mqtt.generic/src/main/java/org/openhab/binding/mqtt/generic/values/NumberValue.java
View File

@ -47,6 +47,7 @@ import org.slf4j.LoggerFactory;
@NonNullByDefault @NonNullByDefault
public class NumberValue extends Value { public class NumberValue extends Value {
private static final String NAN = "NaN"; private static final String NAN = "NaN";
private static final String NEGATIVE_NAN = "-NaN";
private final Logger logger = LoggerFactory.getLogger(NumberValue.class); private final Logger logger = LoggerFactory.getLogger(NumberValue.class);
private final @Nullable BigDecimal min; private final @Nullable BigDecimal min;
@ -120,7 +121,8 @@ public class NumberValue extends Value {
@Override @Override
public Type parseMessage(Command command) throws IllegalArgumentException { public Type parseMessage(Command command) throws IllegalArgumentException {
if (command instanceof StringType && command.toString().equalsIgnoreCase(NAN)) { if (command instanceof StringType
&& (command.toString().equalsIgnoreCase(NAN) || command.toString().equalsIgnoreCase(NEGATIVE_NAN))) {
return UnDefType.UNDEF; return UnDefType.UNDEF;
} }
return parseCommand(command); return parseCommand(command);

2
bundles/org.openhab.binding.mqtt.generic/src/test/java/org/openhab/binding/mqtt/generic/values/ValueTests.java
View File

@ -189,6 +189,8 @@ public class ValueTests {
assertThat(v.parseMessage(new StringType("NaN")), is(UnDefType.UNDEF)); assertThat(v.parseMessage(new StringType("NaN")), is(UnDefType.UNDEF));
assertThat(v.parseMessage(new StringType("nan")), is(UnDefType.UNDEF)); assertThat(v.parseMessage(new StringType("nan")), is(UnDefType.UNDEF));
assertThat(v.parseMessage(new StringType("-NaN")), is(UnDefType.UNDEF));
assertThat(v.parseMessage(new StringType("-nan")), is(UnDefType.UNDEF));
} }
@Test @Test