Skip to content

feat(export): exporter automatiquement patient__identifier avec Patient - #1320

Open
tfi90 wants to merge 2 commits into
developfrom
feat/export-patient-identifier-link
Open

feat(export): exporter automatiquement patient__identifier avec Patient#1320
tfi90 wants to merge 2 commits into
developfrom
feat/export-patient-identifier-link

Conversation

@tfi90

@tfi90 tfi90 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Objectif

Permettre de retrouver les IPP dans l'export, la colonne patient_source_value ayant disparu avec la migration BC3.1.

Fixes #3397

Changements réalisés

La table patient__identifier est ajoutée au payload d'export dès que Patient est sélectionnée, et un libellé le signale sous le bloc Patient. En export regroupé, l'ajout n'a lieu que si Patient est la seule table sélectionnée.

Impacts

Front, exports.

Tests réalisés

Tests unitaires ajoutés sur le service d'export et sur le composant.

Risques

En export regroupé avec plusieurs tables, l'IPP n'est pas joint : patient__identifier ne partage pas de lien hamiltonien avec Patient dès qu'une autre table est présente.

@tfi90
tfi90 marked this pull request as ready for review July 30, 2026 14:47
@tfi90
tfi90 marked this pull request as draft July 30, 2026 15:03
Retire patient__link du périmètre, reporté sur #3418, et n'injecte plus la
sous-table en export regroupé lorsqu'une autre table est sélectionnée, la
jointure sur clé primaire étant alors refusée.
@cesar-richard cesar-richard self-assigned this Aug 5, 2026
@cesar-richard cesar-richard changed the title feat(export): automatically export patient__identifier and patient__link with Patient feat(export): exporter automatiquement patient__identifier avec Patient Aug 5, 2026
@cesar-richard
cesar-richard marked this pull request as ready for review August 5, 2026 12:34
@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants