Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reprendre l'outil de génération de dictionnaire de variable dans DDI AS #87

Open
dzkwsk opened this issue Jun 14, 2024 · 4 comments
Open
Assignees

Comments

@dzkwsk
Copy link

dzkwsk commented Jun 14, 2024

@dzkwsk dzkwsk changed the title Reprendre l'outil de génération de dictionnaire de variable Reprendre l'outil de génération de dictionnaire de variable dans DDI AS Jun 20, 2024
@EmmanuelDemey
Copy link

@FBibonne et @GtanSndil vont expertiser cette demande.

@FBibonne
Copy link
Member

FBibonne commented Jul 5, 2024

Prévoir un refactor du code au passage :

  • OperationsImpl#getCodeBookExportV2 :
    • créer des variables static final avec les chemins des fichiers xsl
    • ne plus utiliser des fichiers temporaires mais des streams (de java.io)
    • DocumentBuilderFactory doit être configuré pour empécher l'accès aux entité XML externes.
    • Refactorer if (checkResult !="OK") (mésutilistaion des String + factorisation du code)
  • OperationsResources
    • typer les RepsonseEntity retournées
    • ne pas convertir en String les fichiers uploadés si ce n'est pas utile (cf usage qui est fait des strings dans OperationsImpl)
    • ne pas écrire des fichiers temporaires
    • ne pas utiliser returnRmesException
    • Utiliser un enum pour isCodeBook

@FBibonne
Copy link
Member

FBibonne commented Jul 5, 2024

la PR 682 de bauhaus back doit être mergée avant

@FBibonne
Copy link
Member

FBibonne commented Jul 5, 2024

@dzkwsk : faut-il déplacer aussi ?

  • POST /operation/codebook/checkCodeBookContent
  • POST /operation/codebook

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

When branches are created from issues, their pull requests are automatically linked.

4 participants