Projet

Général

Profil

Bug #19098

Erreur sur import d'un WF quand variable n° de téléphone renseignée pour envoi SMS

Ajouté par Brice Mallet il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
29 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Lors de l'import d'un WF (celui-ci : https://eservices.mymeaux.fr/backoffice/workflows/11/), j'ai eu une erreur (cf. copie écran), le n° de téléphone ([form_var_telephone]) renseigné dans l'envoi de SMS (https://eservices.mymeaux.fr/backoffice/workflows/11/status/2/items/2/) étant considéré comme un rôle et donc bloque l'import.
Pour effectivement importer le WF, j'ai dû supprimer [form_var_telephone] du WF en test puis le remettre une fois importé dans la prod.

Fichiers

Capture d_écran de 2017-09-29 15-18-03.png (125 ko) Capture d_écran de 2017-09-29 15-18-03.png Brice Mallet, 29 septembre 2017 15:23
0001-workflows-don-t-use-roles-de-serializer-for-SendSMS-.patch (3,11 ko) 0001-workflows-don-t-use-roles-de-serializer-for-SendSMS-.patch Frédéric Péters, 30 septembre 2017 11:45

Révisions associées

Révision 05f0d8e0 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

workflows: don't use roles (de)serializer for SendSMS "to" attribute (#19098)

Historique

#2

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Projet changé de Publik à w.c.s.

Certainement lié au fait qu'on utilise "to" comme variable cible, et l'import s'attend à des rôles ici. (et à voir côté mail quand on utilise "other", et peut-être aussi au moment du dispatch depuis #18586)

#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

à voir côté mail quand on utilise "other"

Côté mail c'est géré :

    def _get_role_id_from_xml(self, elem, charset, include_id=False):
        # override to allow for destination set with computed values.
#4

Mis à jour par Frédéric Péters il y a plus de 6 ans

Voilà pour le SMS. Concernant le dispatch, je viens de créer #19109.

#5

Mis à jour par Thomas Noël il y a plus de 6 ans

Ack.

(et zut, j'avais raté que c'était aussi simple, oublié le "%s_export/init_with_xml")

#6

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 05f0d8e0d099ae6f637eba6926a83a0396944a0e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 30 11:44:48 2017 +0200

    workflows: don't use roles (de)serializer for SendSMS "to" attribute (#19098)
#7

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF