code improvement, from codebeat advice

Author: jalcaras

3rdparty/reolinkapi.class.php

@@ -160,12 +160,17 @@ public function __construct(array $cnxinfo) {
           $this->cnxtype = strtolower($cnxinfo['cnxtype']);
         }
 
-        if ($this->cnxtype == "http") {
-          ($cnxinfo['port'] == "" || !is_numeric($cnxinfo['port'])) ? $this->port = 80 : $this->port = trim($cnxinfo['port']);
+        if ($cnxinfo['port'] == "" || !is_numeric($cnxinfo['port'])) {
+          if ($this->cnxtype == "http") {
+            $this->port = 80;
+          } else {
+            $this->port = 443;
+          }
         } else {
-          ($cnxinfo['port'] == "" || !is_numeric($cnxinfo['port'])) ? $this->port = 443 : $this->port = trim($cnxinfo['port']);
+          $this->port = trim($cnxinfo['port']);
         }
 
+
         $this->user = trim($cnxinfo['username']);
         $this->password = trim($cnxinfo['password']);
         $this->tagtoken = str_replace(".", "", $this->ip);

core/ajax/reolink.ajax.php

@@ -30,11 +30,13 @@
   */
     ajax::init();
 
+    $camera = reolink::byId(init('id'));
+    if (!is_object($camera)) {
+      throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
+    }
+
+
     if (init('action') == 'CreateCMD') {
-        $camera = reolink::byId(init('id'));
-        if (!is_object($camera)) {
-          throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
-        }
         $camera->GetCamNFO(init('id'));
         $res = $camera->loadCmdFromConf(init('id'));
 
@@ -46,10 +48,6 @@
     }
 
     if (init('action') == 'CheckConnexion') {
-        $camera = reolink::byId(init('id'));
-        if (!is_object($camera)) {
-          throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
-        }
         $res = $camera->TryConnect(init('id'));
         if ($res === true) {
           ajax::success();
@@ -59,43 +57,15 @@
     }
 
     if (init('action') == 'CheckDeviceInfo') {
-        $camera = reolink::byId(init('id'));
-        if (!is_object($camera)) {
-          throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
-        }
         $res = $camera->GetCamNFO(init('id'));
         if ($res === true) {
           ajax::success();
         } else {
-          throw new Exception(__('Impossible de se connecter à la caméra ('.$res.')', __FILE__));
+          throw new Exception(__('Impossible d\'obtenir les informations de la caméra ('.$res.')', __FILE__));
         }
     }
 
-    if (init('action') == 'SetCAMConfig') {
-      $camera = reolink::byId(init('id'));
-      if (!is_object($camera)) {
-        throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
-      }
-      $cfgGroup = init('group');
-      if ($cfgGroup == 'EMAIL'){
-        $res = reolink::setEmail(init('id'));
-      }
-      if ($cfgGroup == 'FTP'){
-        $res = reolink::setFTP(init('id'));
-      }
-
-      if ($res == 200) {
-        ajax::success();
-      } else {
-        throw new Exception(__('Echec du paramétrage de la caméra (consultez le log pour plus de détails)', __FILE__));
-      }
-    }
-
     if (init('action') == 'GetCAMConfig') {
-      $camera = reolink::byId(init('id'));
-      if (!is_object($camera)) {
-        throw new Exception(__('Impossible de trouver la caméra : ' . init('id'), __FILE__));
-      }
       $res = reolink::refreshNFO(init('id'));
       ajax::success();
     }

plugin_info/install.php

@@ -25,7 +25,11 @@ function reolink_install() {
 
 // Fonction exécutée automatiquement après la mise à jour du plugin
   function reolink_update() {
+    $plugin = plugin::byId('reolink');
+    $eqLogics = eqLogic::byType($plugin->getId());
+    foreach ($eqLogics as $eqLogic) {
 
+    }
   }
 
 // Fonction exécutée automatiquement après la suppression du plugin