diff --git a/src/main/java/es/bsc/inb/ades/rest/api/services/FileServiceImp.java b/src/main/java/es/bsc/inb/ades/rest/api/services/FileServiceImp.java index 5af3233446a4a71a98ce2da5d52a98071745ea93..d35c7d84677ab06e9b8ed6ce8756714455ff4ddd 100644 --- a/src/main/java/es/bsc/inb/ades/rest/api/services/FileServiceImp.java +++ b/src/main/java/es/bsc/inb/ades/rest/api/services/FileServiceImp.java @@ -184,13 +184,17 @@ public class FileServiceImp implements FileService { errors.add("The following document is incomplete : " + string + ", please review that both files : "+ string +"_annotations.json and " + string + "_documents.json where included."); } - errors.addAll(validateDocumentsInDatabase(Arrays.asList(files))); - - if(!allow_duplicates) { - //solo se esta validando los _documents.json habria que tambien tener en cuenta los annotations. - errors.addAll(validateDocumentsNamesInDatabase(Arrays.asList(files))); + if(errors.size()==0) { + errors.addAll(validateDocumentsInDatabase(Arrays.asList(files))); + if(!allow_duplicates) { + //solo se esta validando los _documents.json habria que tambien tener en cuenta los annotations. + errors.addAll(validateDocumentsNamesInDatabase(Arrays.asList(files))); + } } + + + return errors; } @@ -212,8 +216,7 @@ public class FileServiceImp implements FileService { errors.add("The document " + name + " has a documentId ("+doc.getLong("id")+") that was already processed and is present in the database. A document with the same documentId cannot be processed again."); } } catch (IOException e) { - // TODO Auto-generated catch block - e.printStackTrace(); + errors.add("The document file " + multipartFile + " is not a valid json files for the PretoxTM system."); } }