Bug #27140
mise en commun de fonctions utilitaires suds
0%
Description
Pas encore #13221 mais déjà, contrib.maarch, contrib.seisin_by_email, contrib.teamnet_axel, ils contiennent tous les mêmes deux fonctions recursive_asdict, client_to_jsondict.
Sans juger celles-ci, sans y toucher vu l'absence de tests pour les deux premiers modules, quand même les déplacer pour en garder une seule copie.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Bug #13221: multiplication des soap.py dans les connecteurs (nettoyage nécessaire) ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à En cours
Une autre copie, partielle/éditée côté greco, je l'avale aussi.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch ajouté
- Statut changé de En cours à Solution proposée
Voilà, en gardant celle de greco.py dont le nom et les variables sont plus explicites et en important directement ces nouvelles méthodes dans les views.py des connecteurs.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Development #27144: Connecteur Cart@ds ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Development #27144: Connecteur Cart@ds supprimé
Mis à jour par Frédéric Péters il y a plus de 5 ans
Cela étant ce code n'est plus nécessaire une fois qu'on utilise zeep (au moins pour sudsobject_to_dict), peut-être pas intéressant du coup de taper dans passerelle.soap du code en rapport avec suds.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Cela étant ce code n'est plus nécessaire une fois qu'on utilise zeep (au moins pour sudsobject_to_dict), peut-être pas intéressant du coup de taper dans passerelle.soap du code en rapport avec suds.
Mais passerelle.soap ne fait que du suds, donc je trouve que c'est bon nettoyage quand même. Je serais pour renommer client_to_jsondict en sudsclient_to_jsondict, et comme ça un "grep suds" dans un connecteur nous permettra de savoir s'il y reste encore du suds qui traine.
Ou bien j'ai pas compris ta remarque.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch 0001-misc-move-duplicated-soap-utility-functions-to-main-.patch ajouté
- Statut changé de Solution validée à Solution proposée
Ou bien j'ai pas compris ta remarque.
C'était sur la pertinence du nettoyage alors que le vrai truc serait de tout passer à zeep. Mais oui, faisons-le quand même, et voici le patch qui en plus renomme en sudsclient_to_jsondict.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Impec.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 412c2a1bdfe2d935fb094258a5d6b6284318546a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Oct 9 15:19:50 2018 +0200
misc: move duplicated soap utility functions to main soap.py (#27140)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
misc: move duplicated soap utility functions to main soap.py (#27140)