Improvements after code review

Author: OndroMih

nucleus/common/simple-glassfish-api/src/main/java/org/glassfish/embeddable/GlassFishVariable.java

@@ -18,7 +18,6 @@
 
 import java.util.Arrays;
 import java.util.Map;
-import java.util.Optional;
 import java.util.stream.Collectors;
 
 /**
@@ -129,10 +128,6 @@ public String getSystemPropertyName() {
         return this.sysPropName;
     }
 
-    public Optional<String> getSystemPropertyValue() {
-        return Optional.ofNullable(System.getProperty(getSystemPropertyName()));
-    }
-
     /**
      * The map contains pairs of {@link #getEnvName()} and {@link #getSystemPropertyName()}.
      * When the {@link #getEnvName()} returns null, the mapping is not included.

nucleus/core/kernel/src/main/java/com/sun/enterprise/v3/server/SystemTasksImpl.java

@@ -135,7 +135,7 @@ private void setVersion() {
     private void setSystemPropertiesFromEnv(boolean forceOverwrite) {
         // adding our version of some system properties.
         setProperty(JAVA_ROOT.getSystemPropertyName(), System.getProperty(JAVA_HOME.getSystemPropertyName()), forceOverwrite);
-        if (HOST_NAME.getSystemPropertyValue().isEmpty()) {
+        if (forceOverwrite || null == System.getProperty(HOST_NAME.getSystemPropertyName())) {
             String hostname = "localhost";
             try {
                 // canonical name checks to make sure host is proper
@@ -144,7 +144,7 @@ private void setSystemPropertiesFromEnv(boolean forceOverwrite) {
                 LOG.log(Level.SEVERE, KernelLoggerInfo.exceptionHostname, ex);
             }
             if (hostname != null) {
-                setProperty(HOST_NAME.getSystemPropertyName(), hostname, false);
+                setProperty(HOST_NAME.getSystemPropertyName(), hostname, true);
             }
         }
     }

nucleus/core/kernel/src/main/java/org/glassfish/runnablejar/UberMain.java

@@ -185,7 +185,7 @@ private void executeCommandFromString(String stringCommand) {
             commandParams = new String[split.length - 1];
             for (int i = 1; i < split.length; i++) {
                 commandParams[i - 1] = split[i].trim();
-        }
+            }
         }
         try {
             CommandResult result = commandParams == null