Development #21842
solis: permettre de dire à passerelle de ne pas envoyer certaines parties du messages à Solis
0%
Description
Genre, quand il n'y a pas de conjoint, il ne pas envoyer la partie "conjoint" dans le webservice. Alors bien sûr, on peut faire différents status avec chacun une actions d'appel webservices spécifique dans le workflow wcs, mais les combinaisons vont être nombreuses et on va s'y perdre vite.
Je propose donc l'ajout de clés magique dans l'appel fait à Passerelle : si on a trouve une clé "del:conjoint" égale à True, alors Passerelle n'enverra pas les informations sur le conjoint à Solis.
Fichiers
Révisions associées
solis: do not delete keys while iterating on (#21842)
because of https://www.python.org/dev/peps/pep-3106/
Historique
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-solis-handle-specific-del-keys-to-remove-parts-of-So.patch 0001-solis-handle-specific-del-keys-to-remove-parts-of-So.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
Il me semble que quand on passera en python 3, ce sera l'embrouille de modifier les clés d'un dictionnaire au moment où tu itères dessus (https://www.python.org/dev/peps/pep-3106/).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 6 ans
Pour situer le contexte: dans le même appel du workflow tu penses envoyer la variable del:conjoint: conjoint_prenom is None ?
Mis à jour par Thomas Noël il y a environ 6 ans
- Patch proposed changé de Oui à Non
Emmanuel Cazenave a écrit :
Il me semble que quand on passera en python 3, ce sera l'embrouille de modifier les clés d'un dictionnaire au moment où tu itères dessus (https://www.python.org/dev/peps/pep-3106/).
Ah bonne info, merci, je vais corriger mon code et faire une liste des clés à détruire, et les détruire ensuite.
Mis à jour par Thomas Noël il y a environ 6 ans
Serghei Mihai a écrit :
Pour situer le contexte: dans le même appel du workflow tu penses envoyer la variable
del:conjoint: conjoint_prenom is None?
Ouaip, ça va être du genre « del:conjoint: =not vars().get('form_var_conjoint_nom') » ... C'est effectivement vraiment wcs qui pilote le connecteur, celui-ci reste vraiment une simple "passerelle" pour le coup ; c'est lié au fait que le format JSON attendu par Solis, je suis certain qu'il va changer, bouger, évoluer, voire être différent selon le client.
Bref, c'est pas la super classe ; mais ça se documente.
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0002-solis-do-not-delete-keys-while-iterating-on-21842.patch 0002-solis-do-not-delete-keys-while-iterating-on-21842.patch ajouté
- Patch proposed changé de Non à Oui
Donc en fait, j'ajoute au précédent 0001 le second patch ci-joint, qui corrige aussi les del qui étaient déjà là avant pour la gestion des fichiers ; et le message de commit explique un peu le pourquoi du patch.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 6 ans
Solis n'ignore donc pas des paramètres, dont il n'a pas besoin, comme "del:conjoint", ok.
Je trouve mieux d'avoir un premier patch qui corrige la suppression des clés en file: et un second qui supprimer les del:.
Mis à jour par Thomas Noël il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 8cad29db28e4da1595562e7efc3f0fc0463b84df
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Feb 14 14:26:35 2018 +0100
solis: do not delete keys while iterating on (#21842)
because of https://www.python.org/dev/peps/pep-3106/
commit 3279529ddc2d4d75558af5d05fabc7435e79fdb0
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Feb 14 12:52:24 2018 +0100
solis: handle specific 'del:keys' to remove parts of Solis message (#21842)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
solis: handle specific 'del:keys' to remove parts of Solis message (#21842)