Skip to content

Commit 501d490

Browse files
committed
continuer les modifs
1 parent 5104f43 commit 501d490

3 files changed

Lines changed: 11 additions & 5 deletions

File tree

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -543,7 +543,6 @@ private void processCampaignWithMode(String campaignName, Mode mode, String root
543543
}
544544
} catch (IOException | ParserConfigurationException | SAXException | XMLStreamException e) {
545545

546-
// Wrap technique -> métier
547546
throw new GenesisException(
548547
HttpStatus.INTERNAL_SERVER_ERROR,
549548
"Error while importing campaign %s (mode %s)"

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,16 +93,19 @@ private MetadataModel readMetadatas(String campaignName, String modeName, FileUt
9393
lunaticFilePath = fileUtils.findFile(String.format("%s/%s", fileUtils.getSpecFolder(campaignName), modeName), LUNATIC_FILE_PATTERN);
9494
metadataModel = parseMetadata(lunaticFilePath, ddiFilePath);
9595
} catch (NoSuchFileException e) {
96-
log.warn("Specification file missing for campaign={}, mode={}", campaignName, modeName);
96+
log.debug("Specification file missing for campaign={}, mode={}", campaignName, modeName);
9797
//DDI file not found and already log - Go to next step
9898
} catch (IOException e) {
9999
log.warn("No DDI File found for {}, {} mode. Will try to use Lunatic...", campaignName, modeName);
100100
}
101101
if(metadataModel == null ){
102-
log.warn("DDI not found or error occurred. Trying Lunatic metadata...for {}, {} mode", campaignName, modeName);
102+
log.debug("DDI not found or error occurred. Trying Lunatic metadata...for {}, {} mode", campaignName, modeName);
103103
try {
104104
lunaticFilePath = fileUtils.findFile(String.format("%s/%s", fileUtils.getSpecFolder(campaignName), modeName), LUNATIC_FILE_PATTERN);
105105
return parseMetadata(lunaticFilePath, null);
106+
} catch (NoSuchFileException e) {
107+
log.debug("No Lunatic specification file found for campaign={}, mode={}", campaignName, modeName);
108+
return null;
106109
} catch (Exception ex) {
107110
log.error("Error reading Lunatic metadata file", ex);
108111
errors.add(new GenesisError(ex.toString()));

src/main/java/fr/insee/genesis/exceptions/GenesisExceptionHandler.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,12 @@ public ResponseEntity<String> handleReviewDisabled(ReviewDisabledException excep
6363

6464
@ExceptionHandler(Exception.class)
6565
public ResponseEntity<String> handleAny(Exception exception) {
66-
log.error("Unexpected error (Type: {}) : {}", exception.getClass().getSimpleName(), exception.getMessage(), exception);
67-
return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR)
66+
log.error("Unexpected error (Type: {}) : {}",
67+
exception.getClass().getSimpleName(),
68+
exception.getMessage(),
69+
exception);
70+
return ResponseEntity
71+
.status(HttpStatus.INTERNAL_SERVER_ERROR)
6872
.body("Internal server error");
6973
}
7074
}

0 commit comments

Comments
 (0)