Development #31116
CD13: connecteur MDPH - Ajouter un champs text au retour de l'appel sur le endpoint dossier
100%
Description
Pour pouvoir faire une source de données qui affiche le nom et prénom sur la liste des dossiers il faudrait ajouter un champs text composé du nom puis du prénom.
Fichiers
Révisions associées
mdph13: fix typo (#31116)
mdph13: add a text property to dossiers (fixes #31116)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-mdph13-fix-typo-31116.patch 0002-mdph13-fix-typo-31116.patch ajouté
- Fichier 0003-mdph13-add-a-text-property-to-dossiers-fixes-31116.patch 0003-mdph13-add-a-text-property-to-dossiers-fixes-31116.patch ajouté
- Fichier 0001-mdph13-always-return-filenumber-and-dob-31116.patch 0001-mdph13-always-return-filenumber-and-dob-31116.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 5 ans
Pour montrer que j'ai relu : dans _make_display_name comme tu fais un "if prenom:", le get('prenom', '') final peut être un simple get('prenom')
Je m'interroge sur le "nom.upper()" et la gestion des accents... j'éviterais.
A noter que dans un contexte un peu identique sur le connecteur passerelle/apps/solis, je stocke le text calculé du bénéficiaire dans l'objet Link, comme ça la remontée d'une simple liste des noms des dossiers est super rapide (sinon sur un compte avec 10 dossiers liés, interroger Solis explosait en timeout). Par ailleurs la génération de ce texte est faite via un template django, ce qui permet de moduler d'une instance à l'autre (ici ce n'est que pour CD13, mais bon, sait on jamais, s'ils veulent changer ça peut être fait ; pour info, passerelle/apps/solis/templates/solis/apa_user_text.txt est par défaut :
{{ individu.civilite }} {{ individu.prenom }} {{ individu.nomUsuel }}{% if individu.nomNaissance and individu.nomNaissance != individu.nomUsuel %} ({{ individu.nomNaissance }}){% endif %}
Mis à jour par Thomas Noël il y a environ 5 ans
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Thomas Noël a écrit :
Pour montrer que j'ai relu : dans _make_display_name comme tu fais un "if prenom:", le get('prenom', '') final peut être un simple get('prenom')
Ok, 3 caractères de gagner ! ;)
Je m'interroge sur le "nom.upper()" et la gestion des accents... j'éviterais.
Hmm ? C'est tout unicode, je vire parce qu'en fait c'est inutile ils me renvoient déjà tout en majuscule sans accents je crois.
A noter que dans un contexte un peu identique sur le connecteur passerelle/apps/solis, je stocke le text calculé du bénéficiaire dans l'objet Link, comme ça la remontée d'une simple liste des noms des dossiers est super rapide (sinon sur un compte avec 10 dossiers liés, interroger Solis explosait en timeout). Par ailleurs la génération de ce texte est faite via un template django, ce qui permet de moduler d'une instance à l'autre (ici ce n'est que pour CD13, mais bon, sait on jamais, s'ils veulent changer ça peut être fait ; pour info, passerelle/apps/solis/templates/solis/apa_user_text.txt est par défaut :
[...]
Je ne l'ai pas encore fait ici mais sur atos-genesys je fais simplement du cache sur 1 journée, j'attends de voir les perfs de leur connecteur en vrai pour dire si c'est nécessaire.
Au CD13 on a pas d'état civil complet avec nom de naissance; juste nom/prénom, leur logiciel est un développement interne si j'ai bien compris.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-mdph13-fix-typo-31116.patch 0002-mdph13-fix-typo-31116.patch ajouté
- Fichier 0003-mdph13-add-a-text-property-to-dossiers-fixes-31116.patch 0003-mdph13-add-a-text-property-to-dossiers-fixes-31116.patch ajouté
- Fichier 0001-mdph13-always-return-filenumber-and-dob-31116.patch 0001-mdph13-always-return-filenumber-and-dob-31116.patch ajouté
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit 884854c29e985ad50eea44445c59f4b060ce0ee8.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
mdph13: always return filenumber and dob (#31116)