Project

General

Profile

Project management #72044

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

Added by Marie Kuntz over 1 year ago. Updated over 1 year 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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year ago

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

#9

Updated by Marie Kuntz over 1 year 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