Project

General

Profile

Development #67326

toulouse-maelis: ajout des endpoints pour créer/modifier les responsables légaux

Added by Nicolas Roche 2 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
13 July 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

...


Files


Related issues

Related to Passerelle - Development #66880: toulouse-maelis: ajout des endpoints permettant de créer et de modifier les DUI (ticket chapeau)Nouveau01 July 2022

Actions

Associated revisions

Revision c238a9f2 (diff)
Added by Nicolas Roche 2 months ago

toulouse_maelis: parsifal: add webservices to manage RL (#67326)

History

#1

Updated by Nicolas Roche 2 months ago

On n'a rien pour supprimer le RL2, mais ça ne choque pas Stéphane.

#2

Updated by Nicolas Roche 2 months ago

  • Related to Development #66880: toulouse-maelis: ajout des endpoints permettant de créer et de modifier les DUI (ticket chapeau) added
#3

Updated by Benjamin Dauvergne 2 months ago

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

Ton patern il me me semble qu'il y manque ^/$ pour ne pas matcher les chaînes qui ont ce préfixe (je crois que le souci est généralisé dans le code de passerelle).

La lecture du RL via son id en base plutôt que de dire simplement RL1/RL2 en ignorant son numéro c'est ce qui a été trouvé de plus logique ?

#4

Updated by Nicolas Roche 2 months ago

Ton patern il me me semble qu'il y manque ^/$ pour ne pas matcher les chaînes qui ont ce préfixe

Bien vu, merci.

La lecture du RL via son id en base (...) c'est ce qui a été trouvé de plus logique ?

Oui, c'est pour rester cohérent avec les enfants (et peut-être aussi les personnes autorisées),
où l'on connaîtra l'identifiant de l'enfant depuis le slug de la page combo.

#5

Updated by Nicolas Roche 2 months ago

La lecture du RL via son id en base (...) c'est ce qui a été trouvé de plus logique ?

En fait je pourrais aussi avoir la place de l'enfant dans la liste comme slug de la sous-page combo
(et pour les vaccins je vais effectivement devoir utiliser leur place dans la liste parce qu'ils n'ont pas d'identifiant).
Donc non, pas vraiment de logique, juste que je préfères identifier les RL, enfants et personnes par leur identifiant que par leur place.

J'ajoute plusieurs commits, mais ce ne sont que des ajustements sans conséquence :
  • 0002: modif des payload des tests, parce qu'au début j'étais parti sur les blocs de champs
  • 0003: pour garder en mémoire qu'un bug maélis m'empêche de récupérer les codes d'erreurs
    (je laisse le test car s'il y a correction côté maélis, il améliorera la couverture de code).
    https://redmine.sigec.fr/issues/931
  • 0004: juste du blabla
  • 0005: je n'avais pas compris qu'on pouvait utiliser updateFamily sans passer aucun responsable légal ni enfant.
#6

Updated by Thomas Noël 2 months ago

Sur le premier patch, c'est quand même un peu moche d'avoir du mélange de phrases avec ou pas i18n, d'une ligne à l'autre, genre

        display_category=_('Family'),
        description="Informations sur un responsable légal",

ou
            'NameID': {'description': _('Publik ID')},
            'rl_id': {'description': 'Numéro du représentant légal'},

mais bon, comme le reste est ainsi déjà écrit, let's go :)

Sur les autres :
  • 0002: ok
  • 0003 : je n'ai pas compris d'où vient l'erreur « Unexpected element 'adresse', expected 'lastname' » du test... c'est pas le XML envoyé qui est foireux ? Je m'attendais à ce que le test montre qu'on a reçu un message de UPDATE_FAMILY_ERR
  • 0004 : ligne 308, mette un commentaire et pas une docstring
  • 0005 : "updateFamily sans passer aucun responsable légal ni enfant" : ok mais j'imagine que pour la création c'est quand même nécessaire. Il faut garder ce FAMILY_SCHEMA pour create_family, et créer un UPDATE_FAMILY_SCHEMA pour update_family (genre copy.deepcopy de FAMILY_SCHEMA et del['Oneof'])
#7

Updated by Nicolas Roche 2 months ago

Sur le premier patch, c'est quand même un peu moche ...

Oui, j'ai ajouté un patch au ticket chapeau pour retirer les phrases génériques que j'ai laissées. (#67565)

0003 : je n'ai pas compris d'où vient l'erreur « Unexpected element 'adresse', expected 'lastname' » du test... c'est pas le XML envoyé qui est foireux ? Je m'attendais à ce que le test montre qu'on a reçu un message de UPDATE_FAMILY_ERR

Tout juste, à la base le test est prévu pour montrer qu'on a reçu un message d'erreur de Maélis.
Sauf que la trame renvoyée est refusée par zeep ; j'ai ouvert un ticket sur le redmine Sigec pour avoir une correction (https://redmine.sigec.fr/issues/931).
Je joins un script pour reproduire.

J'ai laissé le test parce qu'une fois le correctif déployé par Sigec il passera dans le code de traitement d'erreur, que j'ai préféré laisser bien qu'aujourd'hui ce soit du code mort.
Tu préfères que je retire tout ça ?

0004 : fait

0005 : j'imagine que pour la création c'est quand même nécessaire.

Oui, tu as raison (pour la création il faut obligatoiremet avoir RL1).

#10

Updated by Thomas Noël 2 months ago

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

Ack

(je n'ai pas bien compris 0003 mais je te fais confiance)

#11

Updated by Nicolas Roche 2 months ago

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

(squashé)

commit c238a9f2b06668959c7c3c1691618025630a758e
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Tue Jul 12 18:56:47 2022 +0200

    toulouse_maelis: parsifal: add webservices to manage RL (#67326)

#12

Updated by Transition automatique 2 months ago

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

Also available in: Atom PDF