Project

General

Profile

Development #39839

DDPACS: impossible d'appeler le connecteur depuis wcs à cause de clauses 'oneOf'

Added by Emmanuel Cazenave 5 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
13 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Cf pièce jointe il y a plusieurs restrictions oneOf dans le schéma qui portent sur des sous objets.

Pas de possiblité dans wcs de décider de passer un objet ou l'autre, on doit tout passer, et coté connecteur genre : "Additional properties are not allowed ('conventionSpecifique' was unexpected)"

Screenshot-2020-2-13 Passerelle.png View (35.8 KB) Emmanuel Cazenave, 13 Feb 2020 02:30 PM

ddpacs.png View (592 KB) Emmanuel Cazenave, 13 Feb 2020 02:49 PM

0001-wip-39839.patch View (2.17 KB) Benjamin Dauvergne, 13 Feb 2020 03:11 PM

0001-mdel_ddpacs-preprocess-create-requests-39839.patch View (2.39 KB) Benjamin Dauvergne, 13 Feb 2020 03:20 PM

0001-mdel_ddpacs-preprocess-create-requests-39839.patch View (2.54 KB) Benjamin Dauvergne, 13 Feb 2020 03:22 PM

41408
41419

Associated revisions

Revision 558fefdf (diff)
Added by Benjamin Dauvergne 5 months ago

mdel_ddpacs: preprocess create requests (#39839)

To help w.c.s.

History

#1 Updated by Benjamin Dauvergne 5 months ago

  • Assignee set to Benjamin Dauvergne
  • Status changed from Nouveau to Rejeté

Passer par une expression python pour générer le bon dico au niveau supérieur du oneOf, ça ne vaut pas le travail que ça génère de faire autrement.

Clé: ../../convention
Valeur: {'conventionSpecifique': form_var_convention_specifique} if form_var_xxx else {...} (Python)

#2 Updated by Emmanuel Cazenave 5 months ago

41419

Je ré-ouvre, il y a trop de oneOF dans le schéma avec des relativement gros objets en dessous pour le contournement python soit praticable (cf capture).

#4 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Nouveau to Solution proposée
  • File 0001-wip-39839.patch View added
  • Tracker changed from Bug to Development
  • Patch proposed changed from No to Yes

Patch en cours d'écriture, j'attends plus de détails pour voir s'il faut gérer les conversions pour les booléens ou les autres oneOf.

#5 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Solution proposée to En cours

#6 Updated by Emmanuel Cazenave 5 months ago

  • Tracker changed from Development to Bug
  • Patch proposed changed from Yes to No
  • Status changed from En cours to Nouveau

Je testais en local, j'ai déplacé ça au SITIV :

Le formulaire : https://demarches-modele-test.demarches.sitiv.fr/backoffice/forms/59/
L'appel webservice : https://demarches-modele-test.demarches.sitiv.fr/backoffice/workflows/27/status/2/items/1/
Le connecteur :https://passerelle-modele-test.demarches.sitiv.fr/mdel-ddpacs/ddpacs/

Une demande en erreur ('ws_send_error_response_err_desc')
https://demarches-modele-test.demarches.sitiv.fr/backoffice/management/demarche-en-ligne-de-preparation-a-la-conclusion-d-un-pacs-ddpacs-2/1/inspect

Et là ça m'en fait une différente (u'FRA' is not of type 'array'), sûrement un autre ticket.

#7 Updated by Benjamin Dauvergne 5 months ago

À voir si ça traite bien tout.

#8 Updated by Benjamin Dauvergne 5 months ago

Traitement de jour, mois, annee Naissance.

#9 Updated by Emmanuel Cazenave 5 months ago

  • Status changed from Solution proposée to Solution validée

Ça a l'air de marcher.

#10 Updated by Emmanuel Cazenave 5 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 558fefdfb5c7c8597e9c43119090e65009c1d496
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Feb 13 15:09:08 2020 +0100

    mdel_ddpacs: preprocess create requests (#39839)

    To help w.c.s

#11 Updated by Frédéric Péters 5 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF