[magentatv] Fix: Start UPNP listener to catch power off button, thing…

… id in log (openhab#11238) * Fix: Start UPNP listener to catch power off button, thing id in log messages Signed-off-by: Markus Michels <markus7017@gmail.com> * comment removed Signed-off-by: Markus Michels <markus7017@gmail.com>
frederictobiasc · Oct 26, 2021 · 28db0e6 · 28db0e6
1 parent 3ef193d
commit 28db0e6
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/...entatv/src/main/java/org/openhab/binding/magentatv/internal/handler/MagentaTVControl.java b/...entatv/src/main/java/org/openhab/binding/magentatv/internal/handler/MagentaTVControl.java
@@ -57,7 +57,7 @@ public MagentaTVControl() {
     }
 
     public MagentaTVControl(MagentaTVDynamicConfig config, MagentaTVNetwork network, HttpClient httpClient) {
-        thingId = config.getFriendlyName();
+        this.thingId = config.getFriendlyName();
         this.network = network;
         this.oauth = new MagentaTVOAuth(httpClient);
         this.config = config;
@@ -71,6 +71,10 @@ public boolean isInitialized() {
         return initialized;
     }
 
+    public void setThingId(String thingId) {
+        this.thingId = thingId;
+    }
+
     /**
      * Returns the thingConfig - the Control class adds various attributes of the
      * discovered device (like the MR model)

diff --git a/...entatv/src/main/java/org/openhab/binding/magentatv/internal/handler/MagentaTVHandler.java b/...entatv/src/main/java/org/openhab/binding/magentatv/internal/handler/MagentaTVHandler.java
@@ -141,6 +141,7 @@ public void initialize() {
     private void initializeThing() {
         String errorMessage = "";
         try {
+            config.setFriendlyName(getThing().getLabel().toString());
             if (config.getUDN().isEmpty()) {
                 // get UDN from device name
                 String uid = this.getThing().getUID().getAsString();
@@ -321,6 +322,7 @@ private void mapKeyToMediateState(String key) {
     protected void connectReceiver() throws MagentaTVException {
         if (control.checkDev()) {
             updateThingProperties();
+            control.setThingId(config.getFriendlyName());
             manager.registerDevice(config.getUDN(), config.getTerminalID(), config.getIpAddress(), this);
             control.subscribeEventChannel();
             control.sendPairingRequest();

diff --git a/...c/main/java/org/openhab/binding/magentatv/internal/network/MagentaTVPoweroffListener.java b/...c/main/java/org/openhab/binding/magentatv/internal/network/MagentaTVPoweroffListener.java
@@ -46,6 +46,7 @@ public class MagentaTVPoweroffListener extends Thread {
     protected final MulticastSocket socket;
     protected @Nullable NetworkInterface networkInterface;
     protected byte[] buf = new byte[256];
+    private boolean started = false;
 
     public MagentaTVPoweroffListener(MagentaTVHandlerFactory handlerFactory,
             @Nullable NetworkInterface networkInterface) throws IOException {
@@ -61,6 +62,7 @@ public MagentaTVPoweroffListener(MagentaTVHandlerFactory handlerFactory,
     public void start() {
         if (!isStarted()) {
             logger.debug("Listening to SSDP shutdown messages");
+            started = true;
             super.start();
         }
     }
@@ -112,18 +114,19 @@ public void run() {
     }
 
     public boolean isStarted() {
-        return socket.isBound();
+        return started;
     }
 
     /**
      * Make sure the socket gets closed
      */
     public void close() {
-        if (isStarted()) {
+        if (started) { // if (isStarted()) {
             logger.debug("No longer listening to SSDP messages");
             if (!socket.isClosed()) {
                 socket.close();
             }
+            started = false;
         }
     }