[jeelink] Do not propagate state changes when sensor things are offli…

…ne. (openhab#2777) Signed-off-by: Volker Bier <[email protected]>
matushromada · Oct 12, 2017 · a099a45 · a099a45
1 parent cfe708f
commit a099a45
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/...ng.jeelink/src/main/java/org/openhab/binding/jeelink/internal/ec3k/Ec3kSensorHandler.java b/...ng.jeelink/src/main/java/org/openhab/binding/jeelink/internal/ec3k/Ec3kSensorHandler.java
@@ -15,6 +15,7 @@
 
 import org.eclipse.smarthome.core.library.types.DecimalType;
 import org.eclipse.smarthome.core.thing.Thing;
+import org.eclipse.smarthome.core.thing.ThingStatus;
 import org.openhab.binding.jeelink.internal.JeeLinkSensorHandler;
 import org.openhab.binding.jeelink.internal.ReadingPublisher;
 import org.openhab.binding.jeelink.internal.RollingAveragePublisher;
@@ -45,7 +46,7 @@ public ReadingPublisher<Ec3kReading> createPublisher() {
         ReadingPublisher<Ec3kReading> publisher = new ReadingPublisher<Ec3kReading>() {
             @Override
             public void publish(Ec3kReading reading) {
-                if (reading != null) {
+                if (reading != null && getThing().getStatus() == ThingStatus.ONLINE) {
                     BigDecimal currentWatt = new BigDecimal(reading.getCurrentWatt()).setScale(1, RoundingMode.HALF_UP);
                     BigDecimal maxWatt = new BigDecimal(reading.getMaxWatt()).setScale(1, RoundingMode.HALF_UP);
 

diff --git a/.../java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java b/.../java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java
@@ -16,6 +16,7 @@
 import org.eclipse.smarthome.core.library.types.DecimalType;
 import org.eclipse.smarthome.core.library.types.OnOffType;
 import org.eclipse.smarthome.core.thing.Thing;
+import org.eclipse.smarthome.core.thing.ThingStatus;
 import org.openhab.binding.jeelink.internal.JeeLinkSensorHandler;
 import org.openhab.binding.jeelink.internal.ReadingPublisher;
 import org.openhab.binding.jeelink.internal.RollingAveragePublisher;
@@ -46,7 +47,7 @@ public ReadingPublisher<LaCrosseTemperatureReading> createPublisher() {
         ReadingPublisher<LaCrosseTemperatureReading> publisher = new ReadingPublisher<LaCrosseTemperatureReading>() {
             @Override
             public void publish(LaCrosseTemperatureReading reading) {
-                if (reading != null) {
+                if (reading != null && getThing().getStatus() == ThingStatus.ONLINE) {
                     BigDecimal temp = new BigDecimal(reading.getTemperature()).setScale(1, RoundingMode.HALF_UP);
 
                     logger.debug(