Bug #8373
Ne pas planter si jamais il y a deux rôles qui arrivent avec le même nom
0%
Description
Quand on était tout seul, on faisait attention à ça; maintenant que des rôles arrivent d'authentic, on devrait se blinder contre ça.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-allow-duplicated-role-names-8373.patch 0001-misc-allow-duplicated-role-names-8373.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Heureusement qu'on a des tests unitaires pour ce genre de truc.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 31510a0f65494892e549881f64ad2cdf209d2b93
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Sep 25 14:44:33 2015 +0200
misc: allow duplicated role names (#8373)
This doesn't let the user create such roles as the result will always be
confusing lists but at least it won't crash if an external process creates
such roles.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Chercher le rôle par id en utilisant l'uuid
- Chercher le rôle par slug en utilisant le slug (reprise après un import-wcs-roles)
- Créer un nouveau rôle avec id <- uuid, name <- name, slug <- uuid
Mis à jour par Frédéric Péters il y a plus de 8 ans
Il n'y a pas l'infra nécessaire pour prendre en charge le changement d'id d'un rôle (aller mettre à jour formulaires, workflows, paramètres avec le nouvel id), pour le moment je ne tenterais donc pas ce changement.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Je n'envisage pas de changer les ids des rôles pré-existants, la modification ne s'appliquera que pour les nouveaux rôles et part du principe qu'il est impossible qu'un rôle existant se retrouve avec un id "entier" ressemblant à un uuid.
Quand un rôle arrive d'authentic2 vers hobo_notify, on recherche d'abord si un rôle avec comme id w.c.s. l'uuid authentic2 existe, sinon on cherche un rôle ayant pour slug w.c.s. l'uuid authentic2 (à vrai dire on peut s'en passer si on corrige la démo et peut-être la recette meaux à la main sur ce point), enfin on recherche un rôle ayant pour slug w.c.s. le slug authentic2 (ceci pour migrer depuis la situation pré-existante utilisant import-wcs-roles). Finalement si ces trois conditions sont absentes, on crée un nouveau rôle ayant pour id w.c.s l'uuid authentic2, poud slug w.c.s le slug authentic2, idem pour le nom. Si un rôle est considéré comme nouveau au même moment par deux processus de hobo_notify au pire l'un des deux gagnera la course mais il n'y aura pas de doublon.
Mis à jour par Frédéric Péters il y a plus de 8 ans
Ok, s'il n'y a pas de changement d'id aux rôles existants, ça ne pose pas de soucis. (ça devrait par contre aller dans un autre ticket, celui-ci est fermé).
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Fermé
misc: allow duplicated role names (#8373)
This doesn't let the user create such roles as the result will always be
confusing lists but at least it won't crash if an external process creates
such roles.