feat(policy): redesign use TLS policy

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

Author: btry

inc/agent.class.php

@@ -1475,7 +1475,7 @@ public function cleanupSubtopics() {
    /**
     * list of topics to cleanup on unenrollment or on enrollment
     *
-    * @return string[]
+    * @return array
     */
    public static function getTopicsToCleanup() {
       $policy = new PluginFlyvemdmPolicy();

inc/policybase.class.php

@@ -69,7 +69,8 @@ abstract class PluginFlyvemdmPolicyBase implements PluginFlyvemdmPolicyInterface
    protected $specificStatuses = [];
 
     /**
-     * get common task statuses
+     * Gets common task statuses
+     *
      * @return array
      */
    public static final function getEnumBaseTaskStatus() {
@@ -89,8 +90,9 @@ public static final function getEnumBaseTaskStatus() {
    }
 
     /**
-     * get specific task statuses
+     * Gets specific task statuses
      * To be overriden in child class
+     *
      * @return array
      */
    public static function getEnumSpecificStatus() {
@@ -109,14 +111,15 @@ public function __construct(PluginFlyvemdmPolicy $policy) {
     * JSON decode properties for the policy and merges them with default values
     * @param string $properties
     * @param array $defaultProperties
+    *
     * @return array
     */
    protected function jsonDecodeProperties($properties, array $defaultProperties) {
       if (empty($properties)) {
          return $defaultProperties;
-      } else {
-         $propertyCollection = json_decode($properties, true);
       }
+
+      $propertyCollection = json_decode($properties, true);
       if (empty($propertyCollection)) {
          return $defaultProperties;
       }
@@ -130,6 +133,7 @@ protected function jsonDecodeProperties($properties, array $defaultProperties) {
     * @param mixed $itemtype
     * @param integer $itemId
     * @param PluginFlyvemdmNotifiableInterface $notifiable
+    *
     * @return bool
     */
    public function canApply($value, $itemtype, $itemId, PluginFlyvemdmNotifiableInterface $notifiable) {
@@ -141,6 +145,7 @@ public function canApply($value, $itemtype, $itemId, PluginFlyvemdmNotifiableInt
     * @param mixed $itemtype
     * @param integer $itemId
     * @param PluginFlyvemdmNotifiableInterface $notifiable
+    *
     * @return boolean
     */
    public function unicityCheck($value, $itemtype, $itemId, PluginFlyvemdmNotifiableInterface $notifiable) {
@@ -161,6 +166,7 @@ public function unicityCheck($value, $itemtype, $itemId, PluginFlyvemdmNotifiabl
     * @param mixed $itemtype
     * @param integer $itemId
     * @param PluginFlyvemdmNotifiableInterface $notifiable
+    *
     * @return boolean
     */
    public function conflictCheck($value, $itemtype, $itemId, PluginFlyvemdmNotifiableInterface $notifiable) {
@@ -171,6 +177,7 @@ public function conflictCheck($value, $itemtype, $itemId, PluginFlyvemdmNotifiab
     * @param mixed $value
     * @param mixed $itemtype
     * @param integer $itemId
+    *
     * @return bool
     */
    public function integrityCheck($value, $itemtype, $itemId) {

inc/policydeployapplication.class.php

@@ -88,6 +88,7 @@ public function integrityCheck($value, $itemtype, $itemId) {
     * @param mixed $value
     * @param mixed $itemtype
     * @param integer $itemId
+    *
     * @return array|boolean
     */
    public function getMqttMessage($value, $itemtype, $itemId) {

inc/policyfactory.class.php

@@ -79,6 +79,10 @@ public function createFromPolicy(PluginFlyvemdmPolicy $policyData) {
             $policy = new PluginFlyvemdmPolicyRemovefile($policyData);
             break;
 
+         case 'm2m':
+            $policy = new PluginFlyvemdmPolicyM2m($policyData);
+            break;
+
          default:
             return null;
       }

inc/policyinteger.class.php

@@ -72,6 +72,7 @@ public function __construct(PluginFlyvemdmPolicy $policy) {
     * @param mixed $value
     * @param mixed $itemtype
     * @param integer $itemId
+    *
     * @return bool
     */
    public function integrityCheck($value, $itemtype, $itemId) {

install/policies/mdm.php

@@ -36,16 +36,21 @@
 $category = 'Mobile Device Management';
 return [
    [
-      'name'                                => __('use TLS', 'flyvemdm'),
-      'symbol'                              => 'useTLS',
+      'name'                                => __('messenging server', 'flyvemdm'),
+      'symbol'                              => 'm2m',
       'group'                               => 'MDM',
-      'type'                                => 'bool',
+      'type'                                => 'm2m',
       'type_data'                           => '',
       'unicity'                             => 1,
       'plugin_flyvemdm_policycategories_id' => $category,
-      'comment'                             => __('use TLS', 'flyvemdm'),
+      'comment'                             => __('messenging server', 'flyvemdm'),
       'default_value'                       => '',
-      'recommended_value'                   => '',
+      'recommended_value'                   => json_encode(
+         [
+            'server' => '',
+            'port'   => '0',
+            'tls'    => '1'
+         ]),
       'is_android_policy'                   => '1',
       'is_android_system'                   => '0',
       'is_apple_policy'                     => '0',