feat(config): lock mqtt prefix setting

Signed-off-by: Thierry Bugier <[email protected]>

Author: btry

inc/agent.class.php

@@ -548,10 +548,6 @@ public function prepareInputForUpdate($input) {
    public function post_addItem() {
       // Notify the agent about its fleets
       $this->updateSubscription();
-
-      if (PluginFlyvemdmConfig::canUpdate()) {
-         // Here we must update mqtt_prefix_locked
-      }
    }
 
    public function post_getFromDB() {

inc/common.class.php

@@ -170,7 +170,7 @@ public static function endsWith($haystack, $needle) {
    }
 
    /**
-    * Removes the mqtt prefix from
+    * Removes the mqtt prefix from a MQTT topic
     *
     * @param string  $topic Topic to modify
     * @param boolean $readConfig set to True to force reading the configuration

inc/config.class.php

@@ -406,6 +406,16 @@ public static function configUpdate($input) {
          }
       }
 
+      //lock mqtt prefix if needed
+      $DbUtil = new DbUtils();
+      $agentsCount = $DbUtil->countElementsInTable('glpi_plugin_flyvemdm_agents');
+      $fleetsCount = $DbUtil->countElementsInTable('glpi_plugin_flyvemdm_fleets');
+      $invitationCount = $DbUtil->countElementsInTable('glpi_plugin_flyvemdm_invitations');
+      if ($agentsCount + $fleetsCount + $invitationCount > 0) {
+         unset($input['mqtt_prefix']);
+         Session::addMessageAfterRedirect(__('Update of MQTT prefix ignored (invitations, agents or fleets created)', 'flyvemdm'), false, WARNING);
+      }
+
       if (isset($_SESSION['plugin_flyvemdm_wizard_step'])) {
          $input = static::processStep($input);
          if (count($input) > 0 && $input !== false) {
@@ -531,11 +541,4 @@ public function isGlpiConfigured() {
 
       return true;
    }
-
-   public function prepareInputForUpdate($input) {
-      if ($input['mqtt_prefix'] != $this->fields['mqtt_prefix'] && $this->fields['mqtt_prefix_locked'] == 0) {
-         $input['mqtt_prefix_locked'] = 1;
-      }
-      return $input;
-   }
 }

inc/fleet.class.php

@@ -315,10 +315,6 @@ public function post_addItem() {
          $topic = $this->getTopic();
          $this->notify("$topic/Policy/$policyName", null, 0, 1);
       }
-
-      if (PluginFlyvemdmConfig::canUpdate()) {
-         // Here we must update mqtt_prefix_locked
-      }
    }
 
    /**

inc/invitation.class.php

@@ -244,10 +244,6 @@ public function post_addItem() {
 
       // Sent invitation email
       $this->sendInvitation();
-
-      if (PluginFlyvemdmConfig::canUpdate()) {
-         // Here we must update mqtt_prefix_locked
-      }
    }
 
    public function post_updateItem($history = 1) {

install/installer.class.php

@@ -601,7 +601,6 @@ protected function createInitialConfig() {
          'mqtt_user'                       => self::BACKEND_MQTT_USER,
          'mqtt_passwd'                     => $MdmMqttPassword,
          'mqtt_prefix'                     => '',
-         'mqtt_prefix_locked'              => '0',
          'instance_id'                     => $instanceId,
          'registered_profiles_id'          => '',
          'guest_profiles_id'               => '',