Projet

Général

Profil

Project management #72044

Erreur sur import d'appli : Erreur lors de la création du rôle

Ajouté par Marie Kuntz il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
05 décembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

J'ai voulu installer une appli sur mon instance, j'ai récupéré l'appli de test ici https://catalogue.publik.love/applications/
À l'import j'ai cette erreur :

Erreur lors de la création du rôle administrateur-des-workflows (400)

Historique

#2

Mis à jour par Marie Kuntz il y a plus d'un an

Et l'appli s'est installée quand même, avec un rôle "administrateur fonctionnel"
https://hobo-mkuntz.test.entrouvert.org/applications/manifest/signalements/

#3

Mis à jour par Frédéric Péters il y a plus d'un an

"administrateur des workflows" ne devrait certainement pas faire partie de l'application.

("administrateur fonctionnel" non plus)

#4

Mis à jour par Frédéric Péters il y a plus d'un an

En fait dans l'application, on a un rôle "Administrateur fonctionnel" avec "administrateur-des-workflows" comme slug,

role_info = {"name": "Administrateur fonctionnel", "slug": "administrateur-des-workflows", "uuid": "86b0ce3e5d3242a09c1e47f3da85556f"}

et quand c'est envoyé on obtient :

{'result': 0,
 'errors': {'__all__': ['Les champs name, ou doivent former un ensemble unique.']}}

parce qu'il y a déjà un rôle avec le nom "Administrateur fonctionnel" dans l'OU (et que sur cette instance son slug n'est pas "administrateur-des-workflows").

#5

Mis à jour par Marie Kuntz il y a plus d'un an

Est-ce qu'on ne passerait pas l'import non fait sous silence, dans ce cas, plutôt que de lever une erreur, alors que le rôle est déjà là ? Ou alors, indiquer que le slug est présent avec tel libellé, ainsi on peut comparer et décider que ce n'est pas grave, ou que c'est grave et qu'on doit intervenir sur le rôle existant.

#6

Mis à jour par Frédéric Péters il y a plus d'un an

alors que le rôle est déjà là

Le rôle n'est que "bizarrement" là, il est là si on considère le nom "Administrateur fonctionnel" mais il n'est pas là si on considère le slug "administrateur-des-workflows". (et si on imagine qu'il peut y avoir des actions qui dépendent du slug du rôle, ce qui est déployé ne marcherait pas).

Pour moi il faut garder l'erreur sur cette situation.

(et corriger l'application qui ne devrait pas distribuer un tel rôle)

#7

Mis à jour par Marie Kuntz il y a plus d'un an

Frédéric Péters a écrit :

Le rôle n'est que "bizarrement" là, il est là si on considère le nom "Administrateur fonctionnel" mais il n'est pas là si on considère le slug "administrateur-des-workflows". (et si on imagine qu'il peut y avoir des actions qui dépendent du slug du rôle, ce qui est déployé ne marcherait pas).

Mais justement, les actions dépendent du slug, donc si le slug est pré-existant, je ne vois pas le problème que ça pose.

#8

Mis à jour par Frédéric Péters il y a plus d'un an

Mais on peut aussi avoir un dispatch qui se base sur le nom du rôle, par exemple.

#9

Mis à jour par Marie Kuntz il y a plus d'un an

  • Statut changé de Nouveau à Rejeté

Frédéric Péters a écrit :

Mais on peut aussi avoir un dispatch qui se base sur le nom du rôle, par exemple.

Admettons. Je ferme.

Formats disponibles : Atom PDF