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)
| @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 ?)
src/main/java/fr/insee/genesis/controller/rest/responses/RawResponseController.java
Show resolved
Hide resolved
| import fr.insee.genesis.domain.model.surveyunit.Mode; | ||
| import fr.insee.genesis.domain.model.surveyunit.SurveyUnitModel; | ||
| import fr.insee.genesis.domain.model.surveyunit.VariableModel; | ||
| import fr.insee.genesis.domain.model.surveyunit.*; |
| import java.util.Map; | ||
| import java.util.Set; | ||
| import java.time.format.DateTimeFormatter; | ||
| import java.util.*; |
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.
|
3 participants
Ajout de services de "reprocess" des données brutes vers le format Genesis.
Logique du reprocess