Produits Entr'ouvert » Passerelle

Serghei Mihai a écrit :

CRUD

CRUD au sens AstreGS.

Benjamin Dauvergne a écrit :

Ça m'a l'air bon, mais c'est des vrais réponses ? AstreGS ne valide pas les IBAN ?

Quand même une remarque de forme, ça aurait plus uniforme que Get/Update/Suppression prennent les mêmes paramètres en query-string pour désigner le RIB (CodeTiers/IdRib).

Benjamin Dauvergne a écrit :

Ça m'a l'air bon, mais c'est des vrais réponses ? AstreGS ne valide pas les IBAN ?

Oui, des vraies réponses mais j'ai modifié les XML pour ne pas y laisser des vraies coordonées bancaires.

Serghei Mihai a écrit :

Branche à jour.

Je ne le constate que maintenant mais le schéma est incomplet non ? J'en avais mis un complet il me semble; et sur Update on a pas de schéma.

Sur Update tu utilise json_loads au lieu de post_data, pourquoi ?

Benjamin Dauvergne a écrit :

Je le constate que maintenant mais le schéma est incomplet non ? J'en avais mis un complet il me semble; et sur Update on a pas de schéma.

Ce que tu as extrait de la WSDL ne correspond pas avec ce que donne la doc, mais surtout avec l'exemple donné par Pascal dans #43150 qui fonctionne.
Je me suis basé sur son exemple.

L'update exige la clé CodeTiers, IdRib et les autres paramètres sont ceux qu'il faut modifier, sans qu'ils soient obligatoires.
A un moment LibelleCourt peut être modifié, à un autre le CodeStatut (de PROPOSE à VALIDE).

Sur Update tu utilise json_loads au lieu de post_data, pourquoi ?

Pour éviter de passer:

            'request_body': {'schema': {'application/json': {}}}

Serghei Mihai a écrit :

Benjamin Dauvergne a écrit :

Je le constate que maintenant mais le schéma est incomplet non ? J'en avais mis un complet il me semble; et sur Update on a pas de schéma.

Ce que tu as extrait de la WSDL ne correspond pas avec ce que donne la doc, mais surtout avec l'exemple donné par Pascal dans #43150 qui fonctionne.

Je ne saisis pas bien: je vois dans l'exemple de Pascal et dans le WSDL, la clé NumeroIban mais dans ton schéma je ne la vois pas, https://git.entrouvert.org/passerelle.git/commit/?h=wip/43119-astregs-implementer-le-web-servi&id=1d7a7ddf40672a8b57fe865bcbcea56faebbea67 ; et dans l'absolu je ne vois pas trop comment Zeep accepterait des clés qui ne seraient pas dans le WSDL. Au niveau des tests ça a l'air bon mais tu passes des clés qui ne sont pas dans le/les schémas.

Ça me va de ne pas créer un schéma qui reprenne tout le WSDL si ça n'a pas d'intérêt, mais la clé NumeroIban ça me parait important (et que le schéma update soit déclaré si on décide d'exposer ce endpoint).

Benjamin Dauvergne a écrit :

Je ne saisis pas bien: je vois dans l'exemple de Pascal et dans le WSDL, la clé NumeroIban mais dans ton schéma je ne la vois pas, https://git.entrouvert.org/passerelle.git/commit/?h=wip/43119-astregs-implementer-le-web-servi&id=1d7a7ddf40672a8b57fe865bcbcea56faebbea67 ;

NumeroIban et CleBic ajoutés.

Au niveau des tests ça a l'air bon mais tu passes des clés qui ne sont pas dans le/les schémas.

    payload = {
        'CleIban': '76',
        'CodeBic': 'CODEBIC',
        'CodeDevise': 'EUR',
        'CodeIso2Pays': 'FR',
        'CodePaiement': '3',
        'CodeStatut': 'PROPOSE',
        'CodeTiers': '487464',
        'LibelleCompteEtranger': 'LABEL',
        'LibelleCourt': 'Test',
        'LibellePays': 'FR',
        'NumeroIban': 'FR76AABBCCDDEEFFGGHHIIJJKKLLM'
    }

ils ne sont pas dans le schéma JSON du endpoint, mais présents dans la WSDL.

Branche à jour.

Benjamin Dauvergne a écrit :

C'est pas du tout raccord avec ce que tu disais, à part les clés requises qui diffèrent (je pense qu'on peut n'en mettre aucune à part ce qui est listé obligatoire dans la doc, visiblement le fait que les élements XML soient requis n'a pas d'influence sur ce qu'il faut passer à zeep) mon schéma initial était complet, tu aurais pu le garder.

J'ai réduit exprès le schéma que tu as extrait car:

• tous les paramètres ne seront pas renseignés lors de appels WS
• ils sont tous de type string car la couche SOAP n'est qu'un wrapper approximatif au dessus d'une autre API interne qui fait des GET en transmettant tous les paramètres en querystring

C'était pour ne pas rajouter des lignes dans le code sans utilité.

Le schéma de création et de mise à jour devraient être identique comme dans le WSDL, pas compris non plus, enfin je valide, de toute façon a besoin que de Creation visiblement.

On s'est dit dans le commenatire #43119#note-7 que pour l'update CodeTiers et IdRib doivent être passés en query string, donc inutile de les demander dans le payload json.