X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=rapp-manager-models%2Fsrc%2Fmain%2Fjava%2Fcom%2Foransc%2Frappmanager%2Fmodels%2Fcsar%2Fvalidator%2FArtifactDefinitionValidator.java;h=082a1564d22feb3bb0c8c010f3d570ba1eb5ba33;hb=7e301b523d66508f8493f6873e36f0d60e8caaf4;hp=ac00e95011a8a8b337d9708ced9d430c0ec0420a;hpb=4526832bbc18aedfe78d4359c6b058f72c458b81;p=nonrtric%2Fplt%2Frappmanager.git

diff --git a/rapp-manager-models/src/main/java/com/oransc/rappmanager/models/csar/validator/ArtifactDefinitionValidator.java b/rapp-manager-models/src/main/java/com/oransc/rappmanager/models/csar/validator/ArtifactDefinitionValidator.java
index ac00e95..082a156 100755
--- a/rapp-manager-models/src/main/java/com/oransc/rappmanager/models/csar/validator/ArtifactDefinitionValidator.java
+++ b/rapp-manager-models/src/main/java/com/oransc/rappmanager/models/csar/validator/ArtifactDefinitionValidator.java
@@ -36,7 +36,7 @@ public class ArtifactDefinitionValidator implements RappValidator {
     private final RappCsarConfigurationHandler rappCsarConfigurationHandler;
     private final RappValidationUtils rappValidationUtils;
     String invalidAsdErrorMsg = "ASD definition in rApp package is invalid.";
-    private static final int VALIDATION_ORDER = 10;
+    private static final int VALIDATION_ORDER = 15;
 
     @Override
     public int getOrder() {
@@ -47,25 +47,16 @@ public class ArtifactDefinitionValidator implements RappValidator {
     public void validate(Object target, Errors errors) {
         MultipartFile multipartFile = (MultipartFile) target;
         String asdLocation = rappValidationUtils.getAsdDefinitionLocation(multipartFile);
-        if (asdLocation != null && !asdLocation.isEmpty() && rappValidationUtils.isFileExistsInCsar(multipartFile,
-                asdLocation)) {
-            try {
-                String asdContent = rappValidationUtils.getFileFromCsar(multipartFile, asdLocation).toString();
-                if(asdContent != null && !asdContent.isEmpty()) {
-                    JsonNode jsonNode = rappCsarConfigurationHandler.getAsdContentNode(asdContent);
-                    List<String> artifactFileList =
-                            jsonNode.at(RappCsarPathProvider.ARTIFACTS_LOCATION_JSON_POINTER).findValuesAsText("file");
-                    artifactFileList.forEach(
-                            artifactFile -> rappValidationUtils.isFileExistsInCsar(multipartFile, artifactFile));
-                } else {
-                    throw new RappValidationException(invalidAsdErrorMsg);
-                }
-            } catch (RappValidationException e) {
-                throw new RappValidationException(e.getMessage());
-            } catch (Exception e) {
-                throw new RappValidationException(invalidAsdErrorMsg);
-            }
-        } else {
+        try {
+            String asdContent = rappValidationUtils.getFileFromCsar(multipartFile, asdLocation).toString();
+            JsonNode jsonNode = rappCsarConfigurationHandler.getAsdContentNode(asdContent);
+            List<String> artifactFileList =
+                    jsonNode.at(RappCsarPathProvider.ASD_ARTIFACTS_LOCATION_JSON_POINTER).findValuesAsText("file");
+            artifactFileList.forEach(
+                    artifactFile -> rappValidationUtils.isFileExistsInCsar(multipartFile, artifactFile));
+        } catch (RappValidationException e) {
+            throw new RappValidationException(e.getMessage());
+        } catch (Exception e) {
             throw new RappValidationException(invalidAsdErrorMsg);
         }
     }