Fixed 3 bugs (#8584)

Fixed 3 bugs:
- Network bridge not listening on bus
- Reconnection handler keeps reconnecting even after a successful reconnection
- Minor typo in alarm wakeup time channels

Signed-off-by: Cedric Boon <cedric.boon@hotmail.com>

diff --git a/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusBridgeHandler.java b/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusBridgeHandler.java
index b50e6fc04a4f4cb6f96ba94db03da9009c2d592a..b8c8879673ef3aea2f66a13c0e3c4823f94f0bfe 100644 (file)
--- a/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusBridgeHandler.java
+++ b/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusBridgeHandler.java
@@ -253,8 +253,9 @@ public abstract class VelbusBridgeHandler extends BaseBridgeHandler {
             int reconnectionInterval = bridgeConfig.reconnectionInterval;
             if (reconnectionInterval > 0) {
                 this.reconnectionHandler = scheduler.scheduleWithFixedDelay(() -> {
-                    if (connect() && reconnectionHandler != null) {
-                        reconnectionHandler.cancel(false);
+                    final ScheduledFuture<?> currentReconnectionHandler = this.reconnectionHandler;
+                    if (connect() && currentReconnectionHandler != null) {
+                        currentReconnectionHandler.cancel(false);
                     }
                 }, reconnectionInterval, reconnectionInterval, TimeUnit.SECONDS);
             }

diff --git a/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusNetworkBridgeHandler.java b/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusNetworkBridgeHandler.java
index dad98ca73d0d20d32b941052bfd5810e1e77ae7b..41db20536e0bde11b0446e742f6b1d45eab7cc02 100644 (file)
--- a/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusNetworkBridgeHandler.java
+++ b/bundles/org.openhab.binding.velbus/src/main/java/org/openhab/binding/velbus/internal/handler/VelbusNetworkBridgeHandler.java
@@ -63,6 +63,13 @@ public class VelbusNetworkBridgeHandler extends VelbusBridgeHandler {
             updateStatus(ThingStatus.ONLINE);
             logger.debug("Bridge online on network address {}:{}", networkBridgeConfig.address,
                     networkBridgeConfig.port);
+
+            // Start Velbus packet listener. This listener will act on all packets coming from
+            // IP-interface.
+            Thread thread = new Thread(this::readPackets, "OH-binding-" + this.thing.getUID());
+            thread.setDaemon(true);
+            thread.start();
+
             return true;
         } catch (IOException ex) {
             onConnectionLost();
@@ -70,12 +77,6 @@ public class VelbusNetworkBridgeHandler extends VelbusBridgeHandler {
                     networkBridgeConfig.port);
         }
 
-        // Start Velbus packet listener. This listener will act on all packets coming from
-        // IP-interface.
-        Thread thread = new Thread(this::readPackets, "OH-binding-" + this.thing.getUID());
-        thread.setDaemon(true);
-        thread.start();
-
         return false;
     }
 

diff --git a/bundles/org.openhab.binding.velbus/src/main/resources/OH-INF/thing/thing-types.xml b/bundles/org.openhab.binding.velbus/src/main/resources/OH-INF/thing/thing-types.xml
index d5f1cd7135c941c401b6c0657c7d4234f6db2783..4b317cdb8d3ef8b5207869d0ea690ec6e7a7628e 100644 (file)
--- a/bundles/org.openhab.binding.velbus/src/main/resources/OH-INF/thing/thing-types.xml
+++ b/bundles/org.openhab.binding.velbus/src/main/resources/OH-INF/thing/thing-types.xml
@@ -1338,7 +1338,7 @@
                                <label>Clock Alarm 1 Wakeup Hour</label>
                        </channel>
                        <channel id="clockAlarm1WakeupMinute" typeId="minute">
-                               <label>Clock Alarm 1 Minute</label>
+                               <label>Clock Alarm 1 Wakeup Minute</label>
                        </channel>
                        <channel id="clockAlarm1BedtimeHour" typeId="hour">
                                <label>Clock Alarm 1 Bedtime Hour</label>