feat(yunohostimporters): no more dumps and standardised callback function to sync

Author: mrflos

services/YunohostCLIAppImporter.php

@@ -126,18 +126,15 @@ public function syncData($data)
         $yunohostAppFunc = static function ($entry1, $entry2) {
             $value1 = isset($entry1['settings']) ? $entry1['settings']['app'] : $entry1['yunohost_app_id'];
             $value2 = isset($entry2['settings']) ? $entry2['settings']['app'] : $entry2['yunohost_app_id'];
-            var_dump('new app', $value1, $value2);
             return $value1 <=> $value2;
         };
         $newYunohostApps = array_udiff($data, $existingEntries, $yunohostAppFunc);
-        var_dump('final new app', $newYunohostApps);
         $removedYunohostApps = array_udiff($existingEntries, $data, $yunohostAppFunc);
-        var_dump('final removed app', $removedYunohostApps);
         foreach ($newYunohostApps as $entry) {
             $entry['antispam'] = 1;
             try {
                 $this->entryManager->create($this->config['formId'], $entry);
-                //echo 'La fiche de l\'application "'.$entry['bf_titre'].'" a bien été créée.'."\n";
+                echo 'La fiche de l\'application "'.$entry['bf_titre'].'" a bien été créée.'."\n";
             } catch (Exception $ex) {
                 echo 'Erreur lors de la création de la fiche application '.$entry['bf_titre'].' : '.$ex->getMessage()."\n";
             }

services/YunohostCLIUserImporter.php

@@ -106,32 +106,37 @@ public function syncFormModel()
     public function syncData(array $data)
     {
         $existingEntries = $this->entryManager->search(['formsIds' => [$this->config['formId']]]);
-        $newYunohostUsers = array_udiff(array_keys($data), array_keys($existingEntries), 'strcasecmp');
-        $removedYunohostUsers = array_udiff(array_keys($existingEntries), array_keys($data), 'strcasecmp');
+        $yunohostUserFunc = static function ($entry1, $entry2) {
+            $value1 = isset($entry1['username']) ? $entry1['username'] : $entry1['bf_titre'];
+            $value2 = isset($entry2['username']) ? $entry2['username'] : $entry2['bf_titre'];
+            return $value1 <=> $value2;
+        };
+        $newYunohostUsers = array_udiff($data, $existingEntries, $yunohostUserFunc);
+        $removedYunohostUsers = array_udiff($existingEntries, $data, $yunohostUserFunc);
         foreach ($newYunohostUsers as $entry) {
-            $data[$entry]['antispam'] = 1;
+            $entry['antispam'] = 1;
             try {
-                $this->entryManager->create($this->config['formId'], $data[$entry]);
-                echo 'L\'utilisateur "'.$data[$entry]['bf_titre'].'" créé.'."\n";
+                $this->entryManager->create($this->config['formId'], $entry);
+                echo 'L\'utilisateur "'.$entry['bf_titre'].'" créé.'."\n";
             } catch (Exception $ex) {
-                echo 'Erreur lors de la création de la fiche utilisateur '.$data[$entry]['bf_titre'].' : '.$ex->getMessage()."\n";
+                echo 'Erreur lors de la création de la fiche utilisateur '.$entry['bf_titre'].' : '.$ex->getMessage()."\n";
             }
         }
         foreach ($removedYunohostUsers as $entry) {
             try {
                 // TODO use this when 4.5 is released
-                // $this->entryManager->delete($existingEntries[$entry]['id_fiche']);
-                $tag = $existingEntries[$entry]['id_fiche'];
+                // $this->entryManager->delete($entry['id_fiche']);
+                $tag = $entry['id_fiche'];
                 $fiche = $this->entryManager->getOne($tag, false, null, true);
                 if (empty($fiche)) {
                     throw new Exception("Not existing entry : $tag");
                 }
                 $this->services->get(PageManager::class)->deleteOrphaned($tag);
                 $this->services->get(TripleStore::class)->delete($tag, TripleStore::TYPE_URI, null, '', '');
                 $this->services->get(TripleStore::class)->delete($tag, TripleStore::SOURCE_URL_URI, null, '', '');
-                echo 'L\'utilisateur "'.$existingEntries[$entry]['bf_titre'].'" a été supprimé.'."\n";
+                echo 'L\'utilisateur "'.$entry['bf_titre'].'" a été supprimé.'."\n";
             } catch (Exception $ex) {
-                echo 'Erreur lors de la suppression de la fiche utilisateur '.$existingEntries[$entry]['bf_titre'].' : '.$ex->getMessage()."\n";
+                echo 'Erreur lors de la suppression de la fiche utilisateur '.$entry['bf_titre'].' : '.$ex->getMessage()."\n";
             }
         }
         return;