Skip to content

Commit 7b987bc

Browse files
committed
modify after CR
1 parent 0094f8a commit 7b987bc

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

src/main/java/fr/insee/genesis/domain/service/metadata/QuestionnaireMetadataService.java

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -44,20 +44,25 @@ public MetadataModel find(String questionnaireId, Mode mode) throws GenesisExcep
4444
return questionnaireMetadataModels.getFirst().metadataModel();
4545
}
4646

47-
public MetadataModel loadAndSaveIfNotExists(String campaignName, String questionnaireId, Mode mode, FileUtils fileUtils,
48-
List<GenesisError> errors) throws GenesisException {
49-
50-
List<QuestionnaireMetadataModel> questionnaireMetadataModels =
47+
public MetadataModel loadAndSaveIfNotExists(
48+
String campaignName,
49+
String questionnaireId,
50+
Mode mode,
51+
FileUtils fileUtils,
52+
List<GenesisError> errors
53+
) throws GenesisException {
54+
55+
List<QuestionnaireMetadataModel> storedMetadatas =
5156
questionnaireMetadataPersistancePort.find(questionnaireId.toUpperCase(), mode);
5257

53-
MetadataModel metadataModel;
54-
55-
if (questionnaireMetadataModels.isEmpty() || questionnaireMetadataModels.getFirst().metadataModel() == null) {
56-
metadataModel = readMetadatas(campaignName, mode.getModeName(), fileUtils, errors);
57-
} else {
58-
metadataModel = questionnaireMetadataModels.getFirst().metadataModel();
58+
if (!storedMetadatas.isEmpty()
59+
&& storedMetadatas.getFirst().metadataModel() != null) {
60+
return storedMetadatas.getFirst().metadataModel();
5961
}
6062

63+
MetadataModel metadataModel =
64+
readMetadatas(campaignName, mode.getModeName(), fileUtils, errors);
65+
6166
saveMetadata(questionnaireId.toUpperCase(), mode, metadataModel);
6267

6368
return metadataModel;

0 commit comments

Comments
 (0)