fix: error code 1 when launching an instance (#1420)

Ladvace · web-flow · commit cdae501a7b56 · 2022-08-09T00:06:41.000+02:00
* fix: add required java args if missing

* fix: replace required args if not missing

* fix
diff --git a/public/electron.js b/public/electron.js
@@ -168,21 +168,6 @@ const userAgent = new UserAgent({
 // app.allowRendererProcessReuse = true;
 Menu.setApplicationMenu(Menu.buildFromTemplate(edit));
 
-let oldLauncherUserData = path.join(app.getPath('userData'), 'instances');
-
-// Read config and eventually use new path
-try {
-  const configFile = fss.readFileSync(
-    path.join(app.getPath('userData'), 'config.json')
-  );
-  const config = JSON.parse(configFile);
-  if (config.settings.instancesPath) {
-    oldLauncherUserData = config.settings.instancesPath;
-  }
-} catch {
-  // Do nothing
-}
-
 app.setPath('userData', path.join(app.getPath('appData'), 'gdlauncher_next'));
 
 let allowUnstableReleases = false;
@@ -219,6 +204,34 @@ if (
   }
 }
 
+// Read config and eventually use new path
+try {
+  const configPath = path.join(app.getPath('userData'), 'config.json');
+  const configFile = fss.readFileSync(configPath);
+  const config = JSON.parse(configFile);
+
+  const requiredArgs = [
+    '-Dfml.ignorePatchDiscrepancies=true',
+    '-Dfml.ignoreInvalidMinecraftCertificates=true'
+  ];
+
+  const javaArgs = config.settings.java.args;
+
+  if (config.settings.java.args) {
+    const cleanedJavaArgs = javaArgs
+      .split(' ')
+      .filter(arg => !requiredArgs.includes(arg))
+      .join(' ');
+
+    if (cleanedJavaArgs.split(' ').length !== javaArgs.split(' ').length) {
+      config.settings.java.args = cleanedJavaArgs;
+      fss.writeFileSync(configPath, JSON.stringify(config));
+    }
+  }
+} catch {
+  // Do nothing
+}
+
 log.log(process.env.REACT_APP_RELEASE_TYPE, app.getVersion());
 
 const get7zPath = async () => {
@@ -553,10 +566,6 @@ ipcMain.handle('getSentryDsn', () => {
   return process.env.SENTRY_DSN;
 });
 
-ipcMain.handle('getOldLauncherUserData', () => {
-  return oldLauncherUserData;
-});
-
 ipcMain.handle('getExecutablePath', () => {
   return path.dirname(app.getPath('exe'));
 });
diff --git a/src/app/desktop/utils/constants.js b/src/app/desktop/utils/constants.js
@@ -6,11 +6,13 @@ export const LINUX = 'linux';
 export const DARWIN = 'darwin';
 export const DESKTOP_PATH = path.join(homedir(), 'Desktop');
 export const CLASSPATH_DIVIDER_CHAR = platform() === WINDOWS ? ';' : ':';
-export const DEFAULT_JAVA_ARGS = `-Dfml.ignorePatchDiscrepancies=true -Dfml.ignoreInvalidMinecraftCertificates=true ${
+export const DEFAULT_JAVA_ARGS = `${
   platform() === WINDOWS
     ? '-XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_minecraft.exe.heapdump'
     : ''
 } -Xms256m`;
+export const REQUIRED_JAVA_ARGS =
+  '-Dfml.ignorePatchDiscrepancies=true -Dfml.ignoreInvalidMinecraftCertificates=true';
 export const DEFAULT_MEMORY = 4096;
 
 export const resolutionPresets = [
diff --git a/src/app/desktop/utils/index.js b/src/app/desktop/utils/index.js
@@ -24,6 +24,7 @@ import {
 import { getAddon, getAddonFile, mcGetPlayerSkin } from '../../../common/api';
 import { downloadFile } from './downloader';
 import browserDownload from '../../../common/utils/browserDownload';
+import { REQUIRED_JAVA_ARGS } from './constants';
 
 export const isDirectory = source =>
   fs.lstat(source).then(r => r.isDirectory());
@@ -495,14 +496,17 @@ export const getJVMArguments112 = (
       .join(process.platform === 'win32' ? ';' : ':')
   );
 
+  const javaArgs = jvmOptions.filter(Boolean);
+
   // if (process.platform === "darwin") {
   //   args.push("-Xdock:name=instancename");
   //   args.push("-Xdock:icon=instanceicon");
   // }
 
   args.push(`-Xmx${memory}m`);
   args.push(`-Xms${memory}m`);
-  args.push(...jvmOptions);
+  args.push(...REQUIRED_JAVA_ARGS.split(' '));
+  if (javaArgs.length > 0) args.push(...javaArgs);
   args.push(
     `-Djava.library.path=${addQuotes(
       needsQuote,
@@ -600,11 +604,11 @@ export const getJVMArguments113 = (
   let args = mcJson.arguments.jvm.filter(v => !skipLibrary(v));
   const needsQuote = process.platform !== 'win32';
 
+  const javaArgs = jvmOptions.filter(Boolean);
   // if (process.platform === "darwin") {
   //   args.push("-Xdock:name=instancename");
   //   args.push("-Xdock:icon=instanceicon");
   // }
-
   args.push(`-Xmx${memory}m`);
   args.push(`-Xms${memory}m`);
   args.push(
@@ -613,7 +617,9 @@ export const getJVMArguments113 = (
   if (mcJson.logging) {
     args.push(mcJson?.logging?.client?.argument || '');
   }
-  args.push(...jvmOptions);
+
+  args.push(...REQUIRED_JAVA_ARGS.split(' '));
+  if (javaArgs.length > 0) args.push(...javaArgs);
 
   // Eventually inject additional arguments (from 1.17 (?))
   if (mcJson?.forge?.arguments?.jvm) {
diff --git a/src/common/components/ModalsManager.js b/src/common/components/ModalsManager.js
@@ -75,9 +75,6 @@ const modalsComponentLookupTable = {
     lazy(() => import('../modals/InstanceCrashed'))
   ),
   ChangeLogs: AsyncComponent(lazy(() => import('../modals/ChangeLogs'))),
-  InstancesMigration: AsyncComponent(
-    lazy(() => import('../modals/InstancesMigration'))
-  ),
   McVersionChanger: AsyncComponent(
     lazy(() => import('../modals/McVersionChanger'))
   ),
diff --git a/src/common/modals/InstancesMigration.js b/src/common/modals/InstancesMigration.js
