diff --git a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceIncludeFilter.java b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceIncludeFilter.java index c25aa181a3d..8b3181a38a7 100644 --- a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceIncludeFilter.java +++ b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceIncludeFilter.java @@ -15,6 +15,7 @@ import java.math.BigDecimal; import org.eclipse.jdt.annotation.NonNullByDefault; +import org.eclipse.jdt.annotation.Nullable; import org.openhab.core.items.Item; import org.openhab.core.library.types.DecimalType; import org.openhab.core.library.types.QuantityType; @@ -38,11 +39,12 @@ public class PersistenceIncludeFilter extends PersistenceFilter { private final String unit; private final boolean inverted; - public PersistenceIncludeFilter(String name, BigDecimal lower, BigDecimal upper, String unit, boolean inverted) { + public PersistenceIncludeFilter(String name, BigDecimal lower, BigDecimal upper, @Nullable String unit, + boolean inverted) { super(name); this.lower = lower; this.upper = upper; - this.unit = unit; + this.unit = (unit == null) ? "" : unit; this.inverted = inverted; } diff --git a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceThresholdFilter.java b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceThresholdFilter.java index 269c40db9d8..1a15070e939 100644 --- a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceThresholdFilter.java +++ b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceThresholdFilter.java @@ -20,6 +20,7 @@ import javax.measure.UnconvertibleException; import org.eclipse.jdt.annotation.NonNullByDefault; +import org.eclipse.jdt.annotation.Nullable; import org.openhab.core.items.Item; import org.openhab.core.library.types.DecimalType; import org.openhab.core.library.types.QuantityType; @@ -47,10 +48,10 @@ public class PersistenceThresholdFilter extends PersistenceFilter { private final transient Map valueCache = new HashMap<>(); - public PersistenceThresholdFilter(String name, BigDecimal value, String unit, boolean relative) { + public PersistenceThresholdFilter(String name, BigDecimal value, @Nullable String unit, boolean relative) { super(name); this.value = value; - this.unit = unit; + this.unit = (unit == null) ? "" : unit; this.relative = relative; } diff --git a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceTimeFilter.java b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceTimeFilter.java index 524ef2bc342..cec6c967744 100644 --- a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceTimeFilter.java +++ b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/filter/PersistenceTimeFilter.java @@ -38,10 +38,10 @@ public class PersistenceTimeFilter extends PersistenceFilter { private transient @Nullable Duration duration; private final transient Map nextPersistenceTimes = new HashMap<>(); - public PersistenceTimeFilter(String name, int value, String unit) { + public PersistenceTimeFilter(String name, int value, @Nullable String unit) { super(name); this.value = value; - this.unit = unit; + this.unit = (unit == null) ? "s" : unit; } public int getValue() {