Fix checkstyle violations in sdc/jtosca
[sdc/sdc-tosca.git] / src / main / java / org / onap / sdc / toscaparser / api / utils / TOSCAVersionProperty.java
index 838fb07..a753d62 100644 (file)
@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -25,104 +25,111 @@ import org.onap.sdc.toscaparser.api.common.JToscaValidationIssue;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
-public class TOSCAVersionProperty {// test with functions/test_concat.yaml
-       
-       private String version;
-       
-       private static final String versionRe =
-                   "^(?<gMajorVersion>([0-9][0-9]*))" +
-            "(\\.(?<gMinorVersion>([0-9][0-9]*)))?" +
-            "(\\.(?<gFixVersion>([0-9][0-9]*)))?" +
-            "(\\.(?<gQualifier>([0-9A-Za-z]+)))?" +
-            "(\\-(?<gBuildVersion>[0-9])*)?$";
-
-       private String minorVersion = null;
-       private String majorVersion = null;
-       private String fixVersion = null;
-       private String qualifier = null;
-       private String buildVersion = null;
-
-       
-       public TOSCAVersionProperty(Object _version) {
-               version = _version.toString();
-
-        if(version.equals("0") || version.equals("0.0") || version.equals("0.0.0")) {
-            //log.warning(_('Version assumed as not provided'))
-            version = "";
+// test with functions/test_concat.yaml
+public class TOSCAVersionProperty {
+
+    private String version;
+
+    private static final String VERSION_RE =
+            "^(?<gMajorVersion>([0-9][0-9]*))"
+                    + "(\\.(?<gMinorVersion>([0-9][0-9]*)))?"
+                    + "(\\.(?<gFixVersion>([0-9][0-9]*)))?"
+                    + "(\\.(?<gQualifier>([0-9A-Za-z]+)))?"
+                    + "(\\-(?<gBuildVersion>[0-9])*)?$";
+
+    private String minorVersion = null;
+    private String majorVersion = null;
+    private String fixVersion = null;
+    private String qualifier = null;
+    private String buildVersion = null;
+
+
+    public TOSCAVersionProperty(String version) {
+
+        if (version.equals("0") || version.equals("0.0") || version.equals("0.0.0")) {
             return;
         }
 
-               Pattern pattern = Pattern.compile(versionRe);
-               Matcher matcher = pattern.matcher(version);
-               if(!matcher.find()) {
-                       ThreadLocalsHolder.getCollector().appendValidationIssue(new JToscaValidationIssue("JE252", String.format(
-                "InvalidTOSCAVersionPropertyException: " +
-                "Value of TOSCA version property \"%s\" is invalid",
-                version))); 
+        Pattern pattern = Pattern.compile(VERSION_RE);
+        Matcher matcher = pattern.matcher(version);
+        if (!matcher.find()) {
+            ThreadLocalsHolder.getCollector().appendValidationIssue(
+                    new JToscaValidationIssue(
+                            "JE252",
+                            "InvalidTOSCAVersionPropertyException: "
+                                    + "Value of TOSCA version property \"" + version + "\" is invalid"
+                    ));
             return;
-               }
+        }
         minorVersion = matcher.group("gMinorVersion");
         majorVersion = matcher.group("gMajorVersion");
         fixVersion = matcher.group("gFixVersion");
-        qualifier = _validateQualifier(matcher.group("gQualifier"));
-        buildVersion = _validateBuild(matcher.group("gBuildVersion"));
-        _validateMajorVersion(majorVersion);
-       
-       }
-       
-       private String _validateMajorVersion(String value) {
+        qualifier = validateQualifier(matcher.group("gQualifier"));
+        buildVersion = validateBuild(matcher.group("gBuildVersion"));
+        validateMajorVersion(majorVersion);
+
+        this.version = version;
+
+    }
+
+    private String validateMajorVersion(String value) {
         // Validate major version
 
         // Checks if only major version is provided and assumes
         // minor version as 0.
         // Eg: If version = 18, then it returns version = '18.0'
 
-        if(minorVersion == null && buildVersion == null && !value.equals("0")) {
+        if (minorVersion == null && buildVersion == null && !value.equals("0")) {
             //log.warning(_('Minor version assumed "0".'))
             version = version + "0";
         }
         return value;
-       }
-
-       private String _validateQualifier(String value) {
-           // Validate qualifier
-       
-           // TOSCA version is invalid if a qualifier is present without the
-           // fix version or with all of major, minor and fix version 0s.
-       
-           // For example, the following versions are invalid
-           //    18.0.abc
-           //    0.0.0.abc
-               
-               if((fixVersion == null && value != null) ||
-                  (minorVersion.equals("0") && majorVersion.equals("0") && 
-                     fixVersion.equals("0") &&  value != null)) {
-                       ThreadLocalsHolder.getCollector().appendValidationIssue(new JToscaValidationIssue("JE253", String.format(
-                       "InvalidTOSCAVersionPropertyException: " +
-                       "Value of TOSCA version property \"%s\" is invalid",
-                       version))); 
-               }
-               return value;
-       }
-
-    private String _validateBuild(String value) {
+    }
+
+    private String validateQualifier(String value) {
+        // Validate qualifier
+
+        // TOSCA version is invalid if a qualifier is present without the
+        // fix version or with all of major, minor and fix version 0s.
+
+        // For example, the following versions are invalid
+        //    18.0.abc
+        //    0.0.0.abc
+
+        if ((fixVersion == null && value != null) || (minorVersion.equals("0") && majorVersion.equals("0")
+                && fixVersion.equals("0") && value != null)) {
+            ThreadLocalsHolder.getCollector().appendValidationIssue(
+                    new JToscaValidationIssue(
+                            "JE253",
+                            "InvalidTOSCAVersionPropertyException: Value of TOSCA version property \""
+                                    + version
+                                    + "\" is invalid"
+                    ));
+        }
+        return value;
+    }
+
+    private String validateBuild(String value) {
         // Validate build version
 
         // TOSCA version is invalid if build version is present without the qualifier.
         // Eg: version = 18.0.0-1 is invalid.
 
-       if(qualifier == null && value != null) {
-                       ThreadLocalsHolder.getCollector().appendValidationIssue(new JToscaValidationIssue("JE254", String.format(
-                       "InvalidTOSCAVersionPropertyException: " +
-                               "Value of TOSCA version property \"%s\" is invalid",
-                               version))); 
-       }
+        if (qualifier == null && value != null) {
+            ThreadLocalsHolder.getCollector().appendValidationIssue(
+                    new JToscaValidationIssue(
+                            "JE254",
+                            "InvalidTOSCAVersionPropertyException: "
+                                    + "Value of TOSCA version property \"" + version + "\" is invalid"
+                    )
+            );
+        }
         return value;
     }
 
-       public Object getVersion() {
-               return version;
-       }
+    public Object getVersion() {
+        return version;
+    }
 
 }