Development #70627
toulouse-maelis: accepter des listes "applaties" depuis w.c.s pour les vaccins
0%
Description
Les items vides posent problème actuellement :
vaccinList/0/code : 'BCG', vaccinList/0/vaccinationDate : '2022-10-24', ... vaccinList/N/code : None vaccinList/N/vaccinationDate : None
Fichiers
Historique
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0002-toulouse-maelis-allow-to-pass-an-empty-vaccin-list-7.patch 0002-toulouse-maelis-allow-to-pass-an-empty-vaccin-list-7.patch ajouté
- Fichier 0003-toulouse-maelis-set-doctor-address-as-optional-70627.patch 0003-toulouse-maelis-set-doctor-address-as-optional-70627.patch ajouté
- Fichier 0001-toulouse-maelis-allow-to-pass-empty-vaccins-to-WS-70.patch 0001-toulouse-maelis-allow-to-pass-empty-vaccins-to-WS-70.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 : accepter des vaccins vides et les retirer de la liste.
0002 : accepter une liste vide de vaccins.
Et j'en profite pour ajouter cette correction :
0003 : rendre l'adresse du docteur optionnelle.
cf https://demo-toulouse.sigec.fr/maelisws-toulouse/doc/family.html#bean-FamilyDoctorBean
Mis à jour par Thomas Noël il y a plus d'un an
0001: dans le test on voit « 'vaccinList/2/code': None, »
Mais d'où vient de "None", de l'appel webservice w.c.s. ? Ca a été testé, parce que c'est pas facile à faire, d'envoyer None, on sait plus facilement envoyer une chaine vide.
0002 : de ce que je comprends, tu supprimes un test pour le remplacer par un autre, c'est un peu dommage... tu veux pas laisser le temps en place et en ajouter un avec params={} ?
0003 : je n'ai pas compris le test, qui ne parle pas de "address" ... je dois rater un truc...?
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de Solution proposée à En cours
Merci Thomas.
Il faut que je reprenne ici, le WSDL maélis ça a bougé depuis.
Mais d'où vient de "None", de l'appel webservice w.c.s.
Ici c'était pour prévoir un formulaire avec les 10 vaccins (pas forcéments remplis),
mais ce n'est plus possible parceque le code du vaccin et devenu obligatoire.
De toute façon c'était plus élégant de passer par getlistdict sur les blocs de champs :
https://demarches-parsifal.test.entrouvert.org/backoffice/management/modifier-information-enfant/231/inspect
{{ form_var_vaccination|getlistdict:"vaccin:code, vaccin_date:vaccinationDate" }}
Ca a été testé, parce que c'est pas facile à faire, d'envoyer None,
On en a dans cette trace :
https://passerelle-parsifal.test.entrouvert.org/manage/toulouse-maelis/test/logs/?log_id=319746
endpoint POST /toulouse-maelis/test/update-coordinate?NameID=a76e9d67c76e45089c66e2372543de43&rl_id=614353&orig=... ('{ "adresse/street2": null, -> champ texte (vide) "adresse/numComp": null, -> champ liste (---)
qui provient de ce formulaire et de cet appel WS :
https://demarches-parsifal.test.entrouvert.org/backoffice/forms/89/fields/
https://demarches-parsifal.test.entrouvert.org/backoffice/workflows/35/status/1/items/1/
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de En cours à Rejeté
Je rejette parce que cette partie est minée (les WS natifs ne permettent que l'ajout) et qu'on s'en sort avec les blocs de champs.
J'ouvrirais un nouveau ticket si besoin avec un cas d'usage bien défini.