Merge pull request #471 from arunvenmany-ibm/variable_allow_empty_fix

allow empty values for variable resolution

Author: venmanyarun

src/main/java/io/openliberty/tools/common/plugins/util/VariableUtility.java

@@ -68,7 +68,7 @@ public static String resolveVariables(CommonLoggerI log, String nodeValue, Colle
         for (String nextVariable : variablesToResolve) {
             String value = getPropertyValue(nextVariable, props, defaultProps, libDirPropFiles);
 
-            if (value == null || value.isEmpty()) {
+            if (value == null) {
                 // Variable could not be resolved. Log message and return null.
                 log.debug("Variable " + nextVariable + " cannot be resolved.");
                 return null;
@@ -183,11 +183,11 @@ public static List<Properties> parseVariables(Document doc, boolean defaultValue
                 String varValue = getValue(attr, "value");
                 String varDefaultValue = getValue(attr, "defaultValue");
 
-                if ((values || both) && (varValue != null && !varValue.isEmpty())) {
+                if ((values || both) && varValue != null) {
                     props.setProperty(varName, varValue);
                 }
 
-                if ((defaultValues || both) && (varDefaultValue != null && ! varDefaultValue.isEmpty())) {
+                if ((defaultValues || both) && varDefaultValue != null) {
                     defaultProps.setProperty(varName, varDefaultValue);
                 }
             }