Project

General

Profile

Gestion de projet #72044

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

Added by Marie Kuntz almost 2 years ago. Updated almost 2 years ago.

Status:
Rejeté
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
05 December 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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)

History

#2

Updated by Marie Kuntz almost 2 years ago

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

Updated by Frédéric Péters almost 2 years ago

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

("administrateur fonctionnel" non plus)

#4

Updated by Frédéric Péters almost 2 years ago

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

Updated by Marie Kuntz almost 2 years ago

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

Updated by Frédéric Péters almost 2 years ago

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

Updated by Marie Kuntz almost 2 years ago

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

Updated by Frédéric Péters almost 2 years ago

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

#9

Updated by Marie Kuntz almost 2 years ago

  • Status changed from Nouveau to 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.

Also available in: Atom PDF