Split loopId into scope/iteration + removed values array to value string

src/main/java/fr/insee/genesis/controller/adapter/LunaticJsonAdapter.java

@@ -13,9 +13,9 @@ public class LunaticJsonAdapter {
 
     public SurveyUnitModel convert(LunaticJsonSurveyUnit su){
         return SurveyUnitModel.builder()
-                .idQuest(su.getIdQuest())
-                .idCampaign("")
-                .idUE(su.getIdUE())
+                .questionnaireId(su.getIdQuest())
+                .campaignId("")
+                .interrogationId(su.getIdUE())
                 .state(DataState.COLLECTED)
                 .mode(Mode.WEB)
                 .recordDate(LocalDateTime.now())

src/main/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapter.java

@@ -69,9 +69,9 @@ public static List<SurveyUnitModel> convert(LunaticXmlSurveyUnit su, VariablesMa
      */
     private static SurveyUnitModel getStateDataFromSurveyUnit(LunaticXmlSurveyUnit su, VariablesMap variablesMap, String idCampaign, DataState dataState, Mode mode) {
         SurveyUnitModel surveyUnitModel = SurveyUnitModel.builder()
-                .idQuest(su.getQuestionnaireModelId())
-                .idCampaign(idCampaign)
-                .idUE(su.getId())
+                .questionnaireId(su.getQuestionnaireModelId())
+                .campaignId(idCampaign)
+                .interrogationId(su.getId())
                 .state(dataState)
                 .mode(mode)
                 .recordDate(LocalDateTime.now())
@@ -119,14 +119,13 @@ private static SurveyUnitModel getCollectedDataFromSurveyUnit(LunaticXmlSurveyUn
                 continue; //Go to next data
             }
             for (int i = 1; i <= valueTypeList.size(); i++) {
-                List<String> variableValues = new ArrayList<>();
                 if (valueTypeList.get(i-1).getValue()!=null) {
-                    variableValues.add(valueTypeList.get(i-1).getValue());
                     variableModels.add(VariableModel.builder()
-                            .idVar(lunaticXmlCollectedData.getVariableName())
-                            .values(variableValues)
-                            .idLoop(LoopIdentifier.getLoopIdentifier(lunaticXmlCollectedData.getVariableName(), variablesMap, i))
-                            .idParent(LoopIdentifier.getRelatedVariableName(lunaticXmlCollectedData.getVariableName(), variablesMap))
+                            .varId(lunaticXmlCollectedData.getVariableName())
+                            .value(valueTypeList.get(i-1).getValue())
+                            .scope(LoopIdentifier.getLoopIdentifier(lunaticXmlCollectedData.getVariableName(), variablesMap))
+                            .parentId(LoopIdentifier.getRelatedVariableName(lunaticXmlCollectedData.getVariableName(), variablesMap))
+                            .iteration(i)
                             .build());
                     dataCount++;
                 }
@@ -157,14 +156,13 @@ private static void getExternalDataFromSurveyUnit(LunaticXmlSurveyUnit su, Surve
                 continue; //Go to next data
             }
             for (int i = 1; i <= valueTypeList.size(); i++) {
-                List<String> variableValues = new ArrayList<>();
                 if (valueTypeList.get(i-1).getValue()!=null) {
-                    variableValues.add(valueTypeList.get(i-1).getValue());
                     variableModels.add(VariableModel.builder()
-                            .idVar(lunaticXmlExternalData.getVariableName())
-                            .values(variableValues)
-                            .idLoop(LoopIdentifier.getLoopIdentifier(lunaticXmlExternalData.getVariableName(), variablesMap, i))
-                            .idParent(LoopIdentifier.getRelatedVariableName(lunaticXmlExternalData.getVariableName(), variablesMap))
+                            .varId(lunaticXmlExternalData.getVariableName())
+                            .value(valueTypeList.get(i-1).getValue())
+                            .scope(LoopIdentifier.getLoopIdentifier(lunaticXmlExternalData.getVariableName(), variablesMap))
+                            .iteration(i)
+                            .parentId(LoopIdentifier.getRelatedVariableName(lunaticXmlExternalData.getVariableName(), variablesMap))
                             .build());
                 }
             }

src/main/java/fr/insee/genesis/controller/dto/SurveyUnitId.java → src/main/java/fr/insee/genesis/controller/dto/InterrogationId.java

@@ -1,6 +1,5 @@
 package fr.insee.genesis.controller.dto;
 
-import com.fasterxml.jackson.annotation.JsonProperty;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
@@ -12,21 +11,20 @@
 @Getter
 @NoArgsConstructor
 @AllArgsConstructor
-public class SurveyUnitId {
+public class InterrogationId {
 
-	@JsonProperty("idUE")
-	private String idUE;
+	private String interrogationId;
 
 	@Override
 	public boolean equals(Object o) {
 		if (this == o) return true;
 		if (o == null || getClass() != o.getClass()) return false;
-		SurveyUnitId that = (SurveyUnitId) o;
-		return Objects.equals(idUE, that.idUE);
+		InterrogationId that = (InterrogationId) o;
+		return Objects.equals(interrogationId, that.interrogationId);
 	}
 
 	@Override
 	public int hashCode() {
-		return Objects.hash(idUE);
+		return Objects.hash(interrogationId);
 	}
 }

src/main/java/fr/insee/genesis/controller/dto/SurveyUnitQualityToolDto.java

@@ -0,0 +1,14 @@
+package fr.insee.genesis.controller.dto;
+
+import lombok.Builder;
+import lombok.Data;
+
+import java.util.List;
+
+@Builder
+@Data
+public class SurveyUnitQualityToolDto {
+    private String surveyUnitId;
+    private List<VariableQualityToolDto> collectedVariables;
+    private List<VariableQualityToolDto> externalVariables;
+}

src/main/java/fr/insee/genesis/controller/dto/SurveyUnitSimplified.java

@@ -11,9 +11,9 @@
 @Data
 public class SurveyUnitSimplified {
 
-	private String idQuest;
-	private String idCampaign;
-	private String idUE;
+	private String questionnaireId;
+	private String campaignId;
+	private String interrogationId;
 	private Mode mode;
 	private List<VariableModel> variablesUpdate;
 	private List<VariableModel> externalVariables;

src/main/java/fr/insee/genesis/controller/dto/VariableDto.java

@@ -10,7 +10,8 @@
 @Data
 public class VariableDto {
     private String variableName;
-    private String idLoop;
+    private String scope;
+    private int iteration;
 
     @JsonProperty("variableStates")
     private List<VariableStateDto> variableStateDtoList;

src/main/java/fr/insee/genesis/controller/dto/VariableInputDto.java

@@ -8,7 +8,7 @@
 @Data
 public class VariableInputDto {
     private String variableName;
-    private String idLoop;
+    private int iteration;
 
     @JsonProperty("newVariableState")
     private VariableStateInputDto variableStateInputDto;

src/main/java/fr/insee/genesis/controller/dto/VariableQualityToolDto.java

@@ -0,0 +1,17 @@
+package fr.insee.genesis.controller.dto;
+
+import com.fasterxml.jackson.annotation.JsonProperty;
+import lombok.Builder;
+import lombok.Data;
+
+import java.util.List;
+
+@Builder
+@Data
+public class VariableQualityToolDto {
+    private String variableName;
+    private Integer iteration;
+
+    @JsonProperty("variableStates")
+    private List<VariableStateDto> variableStateDtoList;
+}

src/main/java/fr/insee/genesis/controller/rest/responses/CampaignController.java

@@ -27,7 +27,7 @@ public CampaignController(SurveyUnitApiPort surveyUnitService) {
     @Operation(summary = "List campaigns in database")
     @GetMapping(path = "/")
     public ResponseEntity<Set<String>> getCampaigns() {
-        Set<String> campaigns = surveyUnitService.findDistinctIdCampaigns();
+        Set<String> campaigns = surveyUnitService.findDistinctCampaignIds();
         return ResponseEntity.ok(campaigns);
     }
 

src/main/java/fr/insee/genesis/controller/rest/responses/IdUEController.java → src/main/java/fr/insee/genesis/controller/rest/responses/InterrogationController.java

@@ -1,6 +1,6 @@
 package fr.insee.genesis.controller.rest.responses;
 
-import fr.insee.genesis.controller.dto.SurveyUnitId;
+import fr.insee.genesis.controller.dto.InterrogationId;
 import fr.insee.genesis.domain.ports.api.SurveyUnitApiPort;
 import io.swagger.v3.oas.annotations.Operation;
 import lombok.extern.slf4j.Slf4j;
@@ -12,23 +12,23 @@
 
 import java.util.List;
 
-@RequestMapping(path = "/idUEs" )
+@RequestMapping(path = "/interrogations" )
 @Controller
 @Slf4j
-public class IdUEController {
+public class InterrogationController {
 
     private final SurveyUnitApiPort surveyUnitService;
 
 
-    public IdUEController(SurveyUnitApiPort surveyUnitService) {
+    public InterrogationController(SurveyUnitApiPort surveyUnitService) {
         this.surveyUnitService = surveyUnitService;
     }
 
 
-    @Operation(summary = "Retrieve all IdUEs for a given questionnaire")
+    @Operation(summary = "Retrieve all interrogations for a given questionnaire")
     @GetMapping(path = "/by-questionnaire")
-    public ResponseEntity<List<SurveyUnitId>> getAllIdUEsByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) {
-        List<SurveyUnitId> responses = surveyUnitService.findDistinctIdUEsByIdQuestionnaire(idQuestionnaire);
+    public ResponseEntity<List<InterrogationId>> getAllInterrogationIdsByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) {
+        List<InterrogationId> responses = surveyUnitService.findDistinctInterrogationIdsByQuestionnaireId(questionnaireId);
         return ResponseEntity.ok(responses);
     }
 

src/main/java/fr/insee/genesis/controller/rest/responses/ModeController.java

@@ -28,15 +28,15 @@ public ModeController(SurveyUnitApiPort surveyUnitService) {
 
     @Operation(summary = "List sources/modes used for a given questionnaire")
     @GetMapping(path = "/by-questionnaire")
-    public ResponseEntity<List<Mode>> getModesByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) {
-        List<Mode> modes = surveyUnitService.findModesByIdQuestionnaire(idQuestionnaire);
+    public ResponseEntity<List<Mode>> getModesByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) {
+        List<Mode> modes = surveyUnitService.findModesByQuestionnaireId(questionnaireId);
         return ResponseEntity.ok(modes);
     }
 
     @Operation(summary = "List sources/modes used for a given campaign")
     @GetMapping(path = "/by-campaign")
-    public ResponseEntity<List<Mode>> getModesByCampaign(@RequestParam("idCampaign") String idCampaign) {
-        List<Mode> modes = surveyUnitService.findModesByIdCampaign(idCampaign);
+    public ResponseEntity<List<Mode>> getModesByCampaign(@RequestParam("campaignId") String campaignId) {
+        List<Mode> modes = surveyUnitService.findModesByCampaignId(campaignId);
         return ResponseEntity.ok(modes);
     }
 

src/main/java/fr/insee/genesis/controller/rest/responses/QuestionnaireController.java

@@ -30,7 +30,7 @@ public QuestionnaireController(SurveyUnitApiPort surveyUnitService) {
     @Operation(summary = "List questionnaires in database")
     @GetMapping(path = "/")
     public ResponseEntity<Set<String>> getQuestionnaires() {
-        Set<String> questionnaires = surveyUnitService.findDistinctIdQuestionnaires();
+        Set<String> questionnaires = surveyUnitService.findDistinctQuestionnaireIds();
         return ResponseEntity.ok(questionnaires);
     }
 
@@ -45,8 +45,8 @@ public ResponseEntity<List<QuestionnaireWithCampaign>> getQuestionnairesWithCamp
 
     @Operation(summary = "List questionnaires used for a given campaign")
     @GetMapping(path = "/by-campaign")
-    public ResponseEntity<Set<String>> getQuestionnairesByCampaign(@RequestParam("idCampaign") String idCampaign) {
-        Set<String> questionnaires = surveyUnitService.findIdQuestionnairesByIdCampaign(idCampaign);
+    public ResponseEntity<Set<String>> getQuestionnairesByCampaign(@RequestParam("campaignId") String campaignId) {
+        Set<String> questionnaires = surveyUnitService.findQuestionnaireIdsByCampaignId(campaignId);
         return ResponseEntity.ok(questionnaires);
     }