Merge pull request #110 from KevinBonnoron/106-lan-browser-gets-nr-to…

…-crash-on-an-uncaught-error 106 lan browser gets nr to crash on an uncaught error
KevinBonnoron · Jan 16, 2024 · 3f65467 · 3f65467
2 parents 27623ee + 9745d0e
commit 3f65467
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 23 deletions.
diff --git a/src/common/base.node.js b/src/common/base.node.js
@@ -2,8 +2,8 @@
 const prettyDate = require('../helpers/date.helper');
 
 const STATUSES = {
-  NOT_CONNECTED: { fill: 'red', shape: 'ring', text: 'not connected' },
-  CONNECTED: { fill: 'green', shape: 'dot', text: 'connected' },
+  NOT_CONNECTED: { fill: 'red', shape: 'ring', text: 'node-red:common.status.not-connected' },
+  CONNECTED: { fill: 'green', shape: 'dot', text: 'node-red:common.status.connected' },
   SERVER_VALIDATION_PENDING: { fill: 'yellow', shape: 'dot', text: 'server validation pending' },
   SERVER_VALIDATION_TIMEOUT: { fill: 'orange', shape: 'dot', text: 'server validation timeout' },
   INVALID_CONFIGURATION: { fill: 'red', shape: 'dot', text: 'invalid configuration' },

diff --git a/src/nodes/connection/connection.js b/src/nodes/connection/connection.js
@@ -8,12 +8,10 @@ module.exports = (RED) => {
       super(config, RED);
     }
 
-    getData(msg) {
+    getData() {
       return {
-        ...msg,
         url: '/connection',
-        method: 'GET',
-        payload: undefined
+        method: 'GET'
       }
     }
   }

diff --git a/src/nodes/freebox-server/freebox-server.js b/src/nodes/freebox-server/freebox-server.js
@@ -46,7 +46,6 @@ module.exports = function (RED) {
       };
 
       this._statusChanged = new EventEmitter();
-      node.on('close', () => this.logout());
 
       this.init(config);
     }
@@ -206,18 +205,6 @@ module.exports = function (RED) {
       });
     }
 
-    /**
-     * logout
-     *
-     * @return void
-     */
-    logout() {
-      this.apiCall('/login/logout', { method: 'POST' }).then(() => {
-        RED.log.info('Session closed');
-        this._statusChanged.emit('session.closed');
-      });
-    }
-
     /**
      * apiCall
      * 

diff --git a/src/nodes/lan-browser/lan-browser.js b/src/nodes/lan-browser/lan-browser.js
@@ -8,12 +8,10 @@ module.exports = (RED) => {
       super(config, RED);
     }
 
-    getData(msg) {
+    getData() {
       return {
-        ...msg,
         url: '/lan/browser/pub',
-        method: 'GET',
-        payload: undefined
+        method: 'GET'
       };
     }
   }