Development #16317
Comportement sur un import référençant un rôle n'existant pas
0%
Description
Pour le moment crash.
Afficher un message disant qu'il manque un rôle (et préciser lequel) ?
Créer la page/cellule sans limiter au rôle qui était dans l'import ?
Fichiers
Révisions associées
general: display an error when importing a site with missing groups (#16317)
Historique
Mis à jour par Thomas Noël il y a presque 7 ans
Frédéric Péters a écrit :
Afficher un message disant qu'il manque un rôle (et préciser lequel) ?
a ma préférence, parce que c'est le même comportement que lors d'un import de wf dans wcs.
Mis à jour par Pierre Cros il y a presque 7 ans
J'avais pas l'impression que ce soit comme ça dans w.c.s. mais oui pour moi la priorité est de faire comme dans w.c.s..
Mis à jour par Anonyme il y a environ 6 ans
Pierre Cros a écrit :
J'avais pas l'impression que ce soit comme ça dans w.c.s. mais oui pour moi la priorité est de faire comme dans w.c.s..
Je me retrouve face à ce bug quand je cherche à importer un site de test d'un collègue. Pour des besoins de développement ou d'exploration du logiciel en tant que dev sur des cas réel, c'est impossible.
Si ça devient urgent, je prendrai le temps d'essayer de régler cette fonctionnalité
Mis à jour par Frédéric Péters il y a environ 6 ans
Ici le comportement ce serait afficher un message d'erreur (précisant le rôle), c'est plutôt accessoire. (le vrai truc important c'est l'export/import de rôles mais c'est ailleurs)
Mis à jour par Frédéric Péters il y a environ 6 ans
En attendant, en local dans le json, suffit de virer les propriétés groups.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0002-general-display-an-error-when-importing-a-site-with-.patch 0002-general-display-an-error-when-importing-a-site-with-.patch ajouté
- Fichier 0001-general-export-groups-as-a-simple-list-of-group-name.patch 0001-general-export-groups-as-a-simple-list-of-group-name.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
En 0001 je me rends compte que le format actuel pour les rôles est une liste de listes contenant le nom [["Rôle 1"], ["Rôle 2"]], je change ça en listes de chaines; c'est avant tout esthétique.
En 0002, le patch touchant directement à ce ticket, avant l'import, vérification de la présence des rôles avant l'import et message d'erreur reprenant les rôles qui manquent, si nécessaire.
Mis à jour par Anonyme il y a environ 6 ans
- Pour 0001: RAS
- Pour 0002: Comme
MissingGroups.namessont utilisées par unprintdans la mgmt-commande et dans un formulaire dans vue, je verrais bien un MissingGroup.__repr__ que factoriserait'Missing groups: %s') % ', '.join(e.names)
Sinon tox => OK
Et Ack !
Mis à jour par Frédéric Péters il y a environ 6 ans
MissingGroup.__repr__
Plutôt unicode si c'est pour aller dans des chaines. (je peux faire ça)
Mis à jour par Anonyme il y a environ 6 ans
Frédéric Péters a écrit :
Plutôt unicode si c'est pour aller dans des chaines. (je peux faire ça)
Ack avec unicode
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
Je pensais avoir attaché un patch avec __unicode__…
commit 18523bc1ffb0da76a76ac84c2a2608528020b9f5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Mar 4 15:51:56 2018 +0100
general: display an error when importing a site with missing groups (#16317)
commit 66c6b4ef011912cc06a4ac005ffd72e5d88c2e64
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Mar 4 15:48:38 2018 +0100
general: export groups as a simple list of group names (#16317)
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
J'ai raté mon intervention mais clairement ici, on pourrait créer les groupes sur la seule base de leur nom, ils seront éventuellement nettoyés au prochain passage d'un hobo_notify de rôles avec full=True (qui n'existe plus là maintenant tout de suite).
Ce mode fonctionnement reste mon objectif tant sur w.c.s. que sur les autres briques (le fait est que le provisionning de rôle des briques Django est déjà en capacité de le faire, coté w.c.s. on attend le #14207), c'est vraiment le plus simple.
Mis à jour par Frédéric Péters il y a environ 6 ans
J'ai raté mon intervention mais clairement ici, on pourrait créer les groupes sur la seule base de leur nom, ils seront éventuellement nettoyés au prochain passage d'un hobo_notify de rôles avec full=True (qui n'existe plus là maintenant tout de suite).
Oui mais pour le moment en tout cas j'ai préféré ne pas inquiéter les agents avec des rôles qui existeraient côté combo et pas authentic. Et plus tard je ne voudrais pas que l'import fonctionne puis que les rôles soient supprimés et que des cellules perdent leur paramétrage.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: export groups as a simple list of group names (#16317)