Bug #41252
affichage erreur import parce que rôle manquant
0%
Description
Sur un import de site, maintenant qu'on passe par des export/import xml pour les formulaires et workflows, il y a validation des rôles, mais l'exception en cas d'erreur n'est pas attrapée.
Stack trace (most recent call first): File "/usr/lib/python3/dist-packages/wcs/workflows.py", line 980, in _get_role_id_from_xml 978 # if the roles are managed by the idp, don't try further. 979 if get_publisher() and get_cfg('sp', {}).get('idp-manage-roles') is True: > 980 raise WorkflowImportError(N_('Unknown referenced role (%s)'), (value,)) 981 982 # and if there's no match, create a new role ... File "/usr/lib/python3/dist-packages/wcs/admin/settings.py", line 1029, in import_submit 1027 1028 def import_submit(self, form): > 1029 return get_publisher().import_zip(form.get_widget('file').parse().fp) 1030 1031 def sitename(self):
Fichiers
Demandes liées
Révisions associées
admin: adjust error message in case of workflow import error (#41252)
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-admin-warn-about-missing-roles-in-site-import-41252.patch 0001-admin-warn-about-missing-roles-in-site-import-41252.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Hésitation ici à afficher un message explicitant qu'Authentic est utilisé, que les rôles doivent commencer par être importés là, etc. mais non, juste un mesage court.
Mis à jour par Thomas Noël il y a presque 4 ans
Ceci étant, de ma lecture, un bout de l'import a pu être fait, on est donc en situation d'un import partiel (sous-entendu : il faut tout effacer et recommencer à zéro).
Je serais pour le dire dans le message, et préciser aussi que c'est pendant l'import d'un workflow que ça a cassé :
msg = _(e) % e.msg_args reason = _('Failed to import a workflow (%s). Only a partial import was made') % msg
C'est un peu moche mais pour l'instant on ne peut pas trop en dire plus, alors c'est déjà ça.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit 735b7cce0a94324e8aa099d1d5eb921af4f5c68c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed May 6 18:40:28 2020 +0200 admin: warn about missing roles in site import (#41252)
+ commit derrière pour ajuster le message selon ta remarque.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Lié à Bug #40369: import de workflow en masse : crash en cas de rôle inexistant ajouté
admin: warn about missing roles in site import (#41252)