Produits Entr'ouvert » Hobo

Quelques lignes plus bas je pense que ceci devient alors inutile :

            # add role's ous
            for user in users:
                for r in user_roles.get(user.id, []):
                    ous.setdefault(r.ou, set()).add(user)

Ensuite, je ne pige pas bien « [None] » dans « for ou in [None] + list(OU.objects.all()): » ... dans le reste du code le parcours des OU est juste fait via OU.objects.all()

Enfin, sur le cas fréquent d'un user qui n'a aucun rôle (compte usager qui vient d'être créé), il me semble qu'on va se retrouver à envoyer autant de notify_agents que d'OUs.

(En vrai c'est mieux si quelqu'un de plus affûté que moi sur le provisionning peut relire)

Quelques lignes plus bas je pense que ceci devient alors inutile :

Sans doute, j'ai fait au plus court.

Ensuite, je ne pige pas bien « [None] »

Parce qu'il y a des utilisateurs qui se trouvent encore avec None comme ou, sans ça les tests ne passent pas.

Enfin, sur le cas fréquent d'un user qui n'a aucun rôle (compte usager qui vient d'être créé), il me semble qu'on va se retrouver à envoyer autant de notify_agents que d'OUs.

Oui, c'est ce qui permet de réfléchir le moins possible.

Quelques lignes plus bas je pense que ceci devient alors inutile :

Sans doute, j'ai fait au plus court.

Désormais supprimées.

Ensuite, je ne pige pas bien « [None] »

Parce qu'il y a des utilisateurs qui se trouvent encore avec None comme ou, sans ça les tests ne passent pas.

Des utilisateurs et/ou des services, et d'ailleurs dans la partie deprovisionning ça passait à côté de ça.

Enfin, sur le cas fréquent d'un user qui n'a aucun rôle (compte usager qui vient d'être créé), il me semble qu'on va se retrouver à envoyer autant de notify_agents que d'OUs.

Oui, c'est ce qui permet de réfléchir le moins possible.

Après y avoir réfléchi un peu quand même je préfère rester ainsi plutôt qu'ajouter une nouvelle branche (rôles avec des attributs (audience = 1 service) / rôles (audience = 1 OU) / pas de rôles (audience = tout)).

Tout ça me semble bien, mais comme je disais plus haut en vrai c'est mieux si quelqu'un d'autre (de plus affûté que moi sur le provisionning) peut relire aussi.

(Et sans doute attendre la prochaine release pour pousser cela au plus tôt)

Sur l'affaire du [None] dans les OU, je note au niveau de hobo/agent/authentic2/management/commands/hobo_provision.py qu'on fait un « ous = {ou.id: ou for ou in get_ou_model().objects.all()} » ... mais je ne sais pas si c'est bien en rapport. (bon, aussi, c'est pas la bonne heure pour relire)