Produits Entr'ouvert » w.c.s.

Point de vue technique : on pourrait penser à un seul champ « à partir d'un gabarit (nameid, email ou id) » qui irait chercher d'abord dans les name_identifier et, si rien n'est trouvé, dans les email. (Prévoir des cas de doublons possibles, dans ce cas prendre le premier utilisateur trouvé, trié par id, et voilà).

Voir ImportFromCsvAfterJob::user_lookup dans wcs/backoffice/data_management.py

(avoir un seul champ était bien mon idée)

Y'a quelque chose qui ne va pas dans le CreateFormdataWorkflowStatusItem::perform je pense:

        elif self.user_association_mode == 'custom' and self.user_association_template:
            with get_publisher().complex_data():
                try:
                    value = self.compute(
                        self.user_association_template,
                        formdata=formdata,
                        raises=True,
                        allow_complex=True,
                        status_item=self,
                    )
                except Exception:
                    # already logged by self.compute
                    pass
                from wcs.variables import LazyUser

                value = get_publisher().get_cached_complex_data(value)

            if isinstance(value, LazyUser):
                value = value._user
            if isinstance(value, get_publisher().user_class):
                new_formdata.user = value
            else:
                new_formdata.user = get_publisher().user_class.lookup_by_string(value)

• le « from wcs.variables import LazyUser » est indenté un peu bizarrement, il faudrait mieux le mettre un cran à gauche, et juste au dessus du isinstance(value, LazyUser) ?
• mais surtout, le value ne va pas exister en cas d'Exception sur le self.compute : il faudrait quitter l'affaire ici, et pas juste faire un "pass"

Pour les libellés moi ça me va, mais j'ajouterais un hint sur le template custom pour dire qu'il « doit renvoyer un objet «User», une adresse électronique ou un UUID (NameID) »

le « from wcs.variables import LazyUser » est indenté un peu bizarrement, il faudrait mieux le mettre un cran à gauche, et juste au dessus du isinstance(value, LazyUser) ?

Ah oui j'ai glissé un peu le code en cours de route.

mais surtout, le value ne va pas exister en cas d'Exception sur le self.compute : il faudrait quitter l'affaire ici, et pas juste faire un "pass"

En effet, je corrige et j'ajoute un test pour cette situation.

(et dans la branche en plus l'ajout du hint).

Proposition de traduction :

Utilisateur associé à la demande :
* Aucun
* Garder l'utilisateur actuel
* Gabarit personnalisé (l'utilisateur associé dépendra de la valeur du gabarit)

Gabarit définissant l'utilisateur à associer (via son courriel, son nameid ou son id)

Utilisateur associé à la demande :

Et si c'est une fiche ?

i.e. son souhaite donc un libellé différent dans ce cas ?

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

Et si c'est une fiche ?
i.e. son souhaite donc un libellé différent dans ce cas ?

Si c'est simple oui c'est un peu mieux mais un seul libellé "Utilisateur associé à la demande/fiche :", ça va très bien aussi.

ou son id

J'imagine que ça vient de nos mentions de nameid/uuid, en fait ces deux éléments sont identiques, mais sont différents de ce qui est réellement l'id de l'usager, par lequel on ne peut pas faire d'association.

La phrase actuelle à ce sujet (en remarque du champ), c'est « Must give a "User" object, an email address or an user UUID (NameID) » (pas sur la capture).

Je pense que je mettrais pas de remarque et que je garderais :

Gabarit définissant l'utilisateur à associer (via son courriel ou son NameID)

Mais s'il faut impérativement une remarque :

Gabarit définissant l'utilisateur à associer
[                                                                              ]
Le gabarit doit renvoyer le NameID ou le courriel de l'utilisateur à associer.