Conversation
src/main/java/fr/insee/genesis/controller/rest/responses/RawResponseController.java
Show resolved
Hide resolved
src/main/java/fr/insee/genesis/controller/rest/responses/RawResponseController.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/insee/genesis/controller/rest/responses/RawResponseController.java
Show resolved
Hide resolved
src/test/java/fr/insee/genesis/stubs/LunaticJsonRawDataPersistanceStub.java
Outdated
Show resolved
Hide resolved
src/test/java/fr/insee/genesis/stubs/RawResponseDataPersistanceStub.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/insee/genesis/domain/service/rawdata/LunaticJsonRawDataService.java
Show resolved
Hide resolved
Hajarel-moukh
force-pushed
the
devReprocessRawDatas
branch
from
b988834 to
9f747cd
Compare
| import org.springframework.web.bind.annotation.RequestMapping; | ||
| import org.springframework.web.bind.annotation.RequestMethod; | ||
| import org.springframework.web.bind.annotation.RequestParam; | ||
| import org.springframework.web.bind.annotation.*; |
There was a problem hiding this comment.
🤔 Je comprends pas ce commentaire : intellij met automatiquement une wildcard à partir d'un certain nombres d'imports d'un même package avec le '"optimize imports".
Si tu trouves que les wildcards sont une mauvaise pratique, on pourrait configurer un prettier sur le projet (autrement on va pas s'embêter à défaire à la main le travail que fait l'ide)
There was a problem hiding this comment.
Justement j'ai config mon IDE pour ne pas avoir ça (Settings/Code style/Java/onglet "Imports"/Class count to use import with '*')
There was a problem hiding this comment.
Ouais mais c'est pas la conf standard de l'ide, dans ce cas il faut mettre un prettier (ou à défaut push un fichier de conf intellij) sur le projet pour que cette conf soit as code
| @Override | ||
| @Deprecated(since = "1.13.0") | ||
| public DataProcessResult processRawData(String campaignName, List<String> interrogationIdList, List<GenesisError> errors) throws GenesisException { | ||
| public DataProcessResult processRawData(String questionnaireId, List<String> interrogationIdList, List<GenesisError> errors) throws GenesisException { |
There was a problem hiding this comment.
Issue: Cette méthode va être supprimée avec le nettoyage
Suggestion: Créer une nouvelle méthode byQuestionnaireId plutôt que modifier une méthode deprecated
There was a problem hiding this comment.
C'est processRawData(String questionnaireId) qu'il faut utiliser c'est ça ?
There was a problem hiding this comment.
Après avoir passé un peu de temps sur la branche c'est toujours pas clair pour moi quelle méthode utiliser pour appeler le "process" des données Lunatic.
Si on veut reprocess les données Lunatic, il faut avoir la méthode qui process, peut-être qu'elle n'était pas à déprécier finalement ? (autrement dit : si c'est pour copier-coller la méthode avec un nom différent, autant la garder tout simplement ?)
There was a problem hiding this comment.
c'est bien processRawData(String questionnaireId), l'ancien se base sur campaignId, ce qui me génait c'est qu'on touchait à une méthode deprecated qui est supprimée dans #437
src/main/java/fr/insee/genesis/controller/rest/responses/RawResponseController.java
Show resolved
Hide resolved
src/main/java/fr/insee/genesis/domain/service/rawdata/LunaticJsonRawDataService.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/insee/genesis/domain/service/rawdata/LunaticJsonRawDataService.java
Outdated
Show resolved
Hide resolved
nsenave
changed the title
There was a problem hiding this comment.
NB : ici au niveau de l'interface la méthode utilise le nom "collectionInstruementId" et l'implémentation "questionnaireId", en m'appuyant sur le fait que c'est la même propriété, juste renommée dans le modèle filière (cf. les @param dans la javadoc des méthodes)
J'ai fait ce compromis pour avoir du polymorphisme et pas dupliquer le code du reprocess service.
Hajarel-moukh
force-pushed
the
devReprocessRawDatas
branch
from
d689ac3 to
79fb1eb
Compare
|
3 participants
Ajout de services de "reprocess" des données brutes vers le format Genesis.
Logique du reprocess