fix nochild recursion

src/ldap/export/group.ts

@@ -198,7 +198,6 @@ export class Group {
         
         // Certains champs nécessitent de petits calculs
         let vals2={};
-
         // ?!
         vals2[ldapConfig.group['password']] = "{CRYPT}"+data['password'];
 
@@ -211,17 +210,15 @@ export class Group {
         }
 
         // Inscription des valeurs calculées par effet de bord
-        if (!await Basics.change("group", gid, "add", vals2)) {
-            throw "Erreur lors de l'ajout des valeurs intelligentes du nouveau groupe.";
-        }
+        if (!await Basics.change("group", gid, "add", vals2)) throw "Erreur lors de l'ajout des valeurs intelligentes du nouveau groupe.";
 
         ["posixGroup", "brGroup"].forEach(cst => {
             let vals3={};
             vals3[ldapConfig.group['classes']]=cst;
             Basics.change("group", gid, "add", vals3).then(res => {
                 if (!res) throw "Erreur lors de l'ajout des valeurs constantes du nouveau groupe.";
-            });         });
-
+            });
+        });
         // Ajout groupes parents et fils
         for (let rel of ["childs","parents"]) {
             data[rel].forEach(gid => {
@@ -231,7 +228,6 @@ export class Group {
                 });
             });
         }
-
         // Utilisation des fonctions adaptées pour assurer la cohérence de l'ensemble
         for (let cat of categories) {
             for (let uid of data[cat]) Tools.add(uid, gid, cat).then(res => {

src/ldap/internal/config.ts

@@ -98,7 +98,6 @@ export class userData extends partUserData {
  * @var {string?} site - Site web du groupe (URL)
  * @var {string} category - Statut du groupe ; binet, section sportive... (actuellement juste 'binet' ou 'free')
  * @var {string[]} parents - Liste des groupes directement parents de celui-ci
- * @var {string[]} childs - Liste des groupes directement enfants de celui-ci
  * @var {string[]} admins - Liste des admins du groupe
  * @var {string[]} speakers - Liste des porte-parole du groupe
  * @var {string[]} members - Liste des membres du groupe
@@ -113,7 +112,6 @@ export class groupData {
     site?: string;
 	category: string;
     parents: string[] = [];
-    //childs: string[] = [];
     admins: string[] = [];
     speakers: string[] = [];
     members: string[] = [];

src/ldap/internal/tools.ts

@@ -85,7 +85,7 @@ export class Tools {
                     // Escape user input
                     val = ldapEscape.filter("${fil}", { fil: val });
                     // Gestion des dns
-                    if (categories.concat(["parents", "child"]).includes(attribute)) {
+                    if (categories.concat(["parents"]).includes(attribute)) {
                         if (domain == "group")  var id = ldapConfig.group.gid;
                         else                    var id = ldapConfig.user.uid;
                         val = id + '=' + val + ',' + ldapConfig[domain].dn;
@@ -122,8 +122,7 @@ export class Tools {
         let dirtyData = {};
         Object.keys(data).forEach(function(key: string) {
             // Some values edit can't change
-            if (![  'admins','speakers','members','followers',
-                    'directory','classes','id','cleanFullName'].includes(key)) dirtyData[dirtyKeys.key]=data[key];
+            if (!categories.concat(['directory','classes','id','cleanFullName']).includes(key)) dirtyData[dirtyKeys.key]=data[key];
         });
         return Basics.change(domain, id, "replace", dirtyData);
     }
@@ -221,22 +220,26 @@ export class Tools {
      * @static
      */
     static async addDFS(uid: string, gid: string, direction: boolean): Promise<boolean> {
+        // Classic DFS
+        var to_visit = [gid];
         // Cas récursif ascendant (admins)
-        if (direction) {
-            var rol = "admins";
-            var tov = ldapConfig.group.childs;
-        }
+        var rol = "admins";
+        async function next_visit(cur_gid) { to_visit.concat(await Basics.searchSingle("group", ldapConfig.group.gid, null, "parents="+cur_gid)); }
         // Cas récursif descendant (membres)
-        else {
+        if (!direction) {
             var rol = "members";
-            var tov = ldapConfig.group.parents;
+            async function next_visit(cur_gid) {
+                // Lecture du champ parents
+                let dns = await Basics.searchSingle("group", ldapConfig.group.parents, cur_gid);
+                // Reformattage DNs en gids de groupes
+                dns.map(dn => dn.split(',')[0].split('=')[1]);
+                to_visit.concat(dns);
+            }
         }
-        // Classic DFS
-        var to_visit = [gid];
         while (to_visit.length > 0) {
             let cur_gid = to_visit.pop();
             Tools.addIfAbsent(uid, "user", cur_gid, "group", rol);
-            to_visit.concat(await Basics.searchSingle("group", tov, cur_gid));
+            await next_visit(cur_gid);
         }
         return true;
     }
@@ -351,27 +354,31 @@ export class Tools {
      * @static
      */
     static async remDFS(uid: string, gid: string, direction: boolean): Promise<boolean> {
+        // Classic DFS
+        var to_visit = [gid];
         // Cas récursif ascendant (admins)
-        if (direction) {
-            var rol = "admins";
-            var tov = ldapConfig.group.childs;
-            var checks = ["admins"];
-        }
-        // Cas récursif descendant (members)
-        else {
-            var rol = "members";
-            var tov = ldapConfig.group.parents;
+        var checks = ["admins"];
+        // Recherche des groupes avec parents qui correspondent (efficace car index)
+        async function next_visit(cur_gid) { to_visit.concat(await Basics.searchSingle("group", ldapConfig.group.gid, null, "parents="+cur_gid)); }
+        // Cas récursif descendant (membres)
+        if (!direction) {
             var checks = ["admins","speakers","members"];
+            async function next_visit(cur_gid) {
+                // Lecture du champ parents
+                let dns = await Basics.searchSingle("group", ldapConfig.group.parents, cur_gid);
+                // Reformattage des dns en gids
+                dns.map(dn => dn.split(',')[0].split('=')[1]);
+                to_visit.concat(dns);
+            }
         }
-        // Classic DFS
-        var to_visit = [gid];
+        var rol = checks[checks.length-1];
         while (to_visit.length > 0) {
             let cur_gid = to_visit.pop();
             // Si le statut de uid dans cur_gid est un statut strict on arrête de boucler
-            for (let cat of checks) if ((await Tools.get(gid, "group", cat)).includes(uid)) continue;
+            for (let cat of checks) if ((await Tools.get(cur_gid, "group", cat)).includes(uid)) continue;
             // Sinon on le tue et on boucle
             Tools.remIfPresent(uid, "user", cur_gid, "group", rol);
-            to_visit.concat(await Basics.searchSingle("group", tov, cur_gid));
+            await next_visit(cur_gid);
         }
         return true;
     }