Order the launch arguments better.

Classpath -> JVM -> Main Class -> Liteloader -> Minecraft -> Client Options -> Mod list
dscalzi · Feb 15, 2019 · c834ca9 · c834ca9
1 parent 6b96770
commit c834ca9
Showing 1 changed file with 42 additions and 31 deletions.
diff --git a/app/assets/js/processbuilder.js b/app/assets/js/processbuilder.js
@@ -241,20 +241,26 @@ class ProcessBuilder {
      */
     constructJVMArguments(mods, tempNativePath){
 
-        let args = ['-Xmx' + ConfigManager.getMaxRAM(),
-            '-Xms' + ConfigManager.getMinRAM(),
-            '-Djava.library.path=' + tempNativePath,
-            '-cp',
-            this.classpathArg(mods, tempNativePath).join(process.platform === 'win32' ? ';' : ':'),
-            this.forgeData.mainClass]
+        let args = []
 
+        // Classpath Argument
+        args.push('-cp')
+        args.push(this.classpathArg(mods, tempNativePath).join(process.platform === 'win32' ? ';' : ':'))
+
+        // Java Arguments
         if(process.platform === 'darwin'){
-            args.unshift('-Xdock:name=WesterosCraft')
-            args.unshift('-Xdock:icon=' + path.join(__dirname, '..', 'images', 'minecraft.icns'))
+            args.push('-Xdock:name=WesterosCraft')
+            args.push('-Xdock:icon=' + path.join(__dirname, '..', 'images', 'minecraft.icns'))
         }
+        args.push('-Xmx' + ConfigManager.getMaxRAM())
+        args.push('-Xms' + ConfigManager.getMinRAM())
+        args = args.concat(ConfigManager.getJVMOptions())
+        args.push('-Djava.library.path=' + tempNativePath)
 
-        args.splice(2, 0, ...ConfigManager.getJVMOptions())
+        // Main Java Class
+        args.push(this.forgeData.mainClass)
 
+        // Forge Arguments
         args = args.concat(this._resolveForgeArgs())
 
         return args
@@ -298,7 +304,7 @@ class ProcessBuilder {
                         val = this.authUser.accessToken
                         break
                     case 'user_type':
-                        val = 'MOJANG'
+                        val = 'mojang'
                         break
                     case 'version_type':
                         val = this.versionData.type
@@ -309,38 +315,43 @@ class ProcessBuilder {
                 }
             }
         }
+
+        // Autoconnect to the selected server.
+        if(ConfigManager.getAutoConnect() && this.server.isAutoConnect()){
+            const serverURL = new URL('my://' + this.server.getAddress())
+            mcArgs.push('--server')
+            mcArgs.push(serverURL.hostname)
+            if(serverURL.port){
+                mcArgs.push('--port')
+                mcArgs.push(serverURL.port)
+            }
+        }
+
+        // Prepare game resolution
+        if(ConfigManager.getFullscreen()){
+            mcArgs.push('--fullscreen')
+            mcArgs.push(true)
+        } else {
+            mcArgs.push('--width')
+            mcArgs.push(ConfigManager.getGameWidth())
+            mcArgs.push('--height')
+            mcArgs.push(ConfigManager.getGameHeight())
+        }
+
+        // Mod List File Argument
         mcArgs.push('--modListFile')
         mcArgs.push('absolute:' + this.fmlDir)
 
+        // LiteLoader
         if(this.usingLiteLoader){
             mcArgs.push('--modRepo')
             mcArgs.push(this.llDir)
 
+            // Set first arg to liteloader tweak class
             mcArgs.unshift('com.mumfrey.liteloader.launch.LiteLoaderTweaker')
             mcArgs.unshift('--tweakClass')
         }
 
-        // Prepare game resolution
-        if(ConfigManager.getFullscreen()){
-            mcArgs.unshift('--fullscreen')
-        } else {
-            mcArgs.unshift(ConfigManager.getGameWidth())
-            mcArgs.unshift('--width')
-            mcArgs.unshift(ConfigManager.getGameHeight())
-            mcArgs.unshift('--height')
-        }
-
-        // Prepare autoconnect
-        if(ConfigManager.getAutoConnect() && this.server.isAutoConnect()){
-            const serverURL = new URL('my://' + this.server.getAddress())
-            mcArgs.unshift(serverURL.hostname)
-            mcArgs.unshift('--server')
-            if(serverURL.port){
-                mcArgs.unshift(serverURL.port)
-                mcArgs.unshift('--port')
-            }
-        }
-
         return mcArgs
     }