Produits Entr'ouvert » w.c.s.

Tu peux reproduire ça sur une instance où il y aurait juste à cliquer sur un bouton pour constater le soucis ?

Qu'est-ce qui ne convient pas avec cette instance ? (elle est chez nous)

Sur https://demarches-cnil.test.entrouvert.org/backoffice/management/mku-formulaire-pour-tester-wf/3/ je n'ai pas de bouton.

Je t'ai ajouté les rôles "agent SRP" (pour voir le bouton "demander des commentaires") et "debug Marie" (pour avoir les boutons de debug). J'ai remis la démarche au statut "A traiter"

Avec le rôle "Agent SRP" je ne reproduis pas le bug en cliquant sur "Demande complément".

Le bug est visible ici :
https://demarches-cnil.test.entrouvert.org/backoffice/workflows/24/logged-errors/20200127-143011-22-24-new-1-erreur-a-levaluation-de-la-condition-TypeError-int-argument-must-be-a-string-a-bytes-like-object-or-a-number-not-nonetype-/
et non sur la demande traitée

Marie Kuntz a écrit :

Le bug est visible ici :
https://demarches-cnil.test.entrouvert.org/backoffice/workflows/24/logged-errors/20200127-143011-22-24-new-1-erreur-a-levaluation-de-la-condition-TypeError-int-argument-must-be-a-string-a-bytes-like-object-or-a-number-not-nonetype-/
et non sur la demande traitée

Hmm ok je m'attendais à une trace à l'écran, le bug est bien là. Tu penses que tu pourrais simplifier le workflow un peu plus ?

Je ne peux pas simplifier plus, là :D : https://demarches-cnil.test.entrouvert.org/backoffice/workflows/24/
Nouveau formulaire, simplifié lui aussi :
https://demarches-cnil.test.entrouvert.org/plaintes/mku-formulaire-simple-pour-tester-wf/
avec le listing : https://demarches-cnil.test.entrouvert.org/backoffice/management/mku-formulaire-simple-pour-tester-wf/

Dans FormData.get_action_roles() je vois déjà un :

            with get_publisher().substitutions.temporary_feed(self):
                if not item.check_condition(self):
                    continue

Je ne sais pas pourquoi tu parles de get_action_roles(), cette méthode n'est pas appelée dans cette vue, cette vue concerne la barre latérale, où il y a évaluation des différentes demandes exposées, pour voir si elles doivent être "verrouillée" ou pas.

Ben j'ai cherché ce qui pouvait demander l'évaluation d'une expression et donc un temporary_feed à cet endroit :

                with get_publisher().substitutions.temporary_feed(user_formdata):
                    if user_roles.intersection(user_formdata.actions_roles): <- ici accès à FormData.actions_roles
                        session.mark_visited_object(user_formdata)

    def get_actions_roles(self):
        if self.is_draft():
            return []

        wf_status = self.get_status()
        if not wf_status:
            return []

        from wcs.workflows import get_role_translation
        status_action_roles = set()
        for item in wf_status.items or []:
            if not hasattr(item, 'by') or not item.by:
                continue
            with get_publisher().substitutions.temporary_feed(self):
                if not item.check_condition(self): <-- ICI
                    continue
            for role in item.by:
                if role == '_submitter':
                    status_action_roles.add(role)
                else:
                    real_role = get_role_translation(self, role)
                    if real_role:
                        status_action_roles.add(real_role)
        return status_action_roles

    actions_roles = property(get_actions_roles)