From 9c6ac295189acab7303952725755dfac87e18470 Mon Sep 17 00:00:00 2001
From: 0neGal <mail@0negal.com>
Date: Mon, 24 Jan 2022 18:21:13 +0100
Subject: no alert when manually changing the game path

When changing the game path by clicking the button you shouldn't be told
it can't find the game and you've to select one manually, as you know
that already. More importantly, if it could be found automatically it'll
just not do anything.

With this change I also changed the "gui.setpath" string to be more
logical, and to make it clear what it does.
---
 src/utils.js | 30 ++++++++++++++++--------------
 1 file changed, 16 insertions(+), 14 deletions(-)

(limited to 'src/utils.js')

diff --git a/src/utils.js b/src/utils.js
index ef647fe..fb94b92 100644
--- a/src/utils.js
+++ b/src/utils.js
@@ -116,7 +116,7 @@ northstar_auto_updates: {
 //
 // If running with CLI it takes in the --setpath argument otherwise it
 // open the systems file browser for the user to select a path.
-async function setpath(win) {
+async function setpath(win, forcedialog) {
 	function setGamepath(folder) {
 		settings.gamepath = folder;
 		settings.zip = path.join(settings.gamepath + "/northstar.zip");
@@ -128,21 +128,23 @@ async function setpath(win) {
 	if (! win) { // CLI
 		setGamepath(cli.param("setpath"));
 	} else { // GUI
-		function setGamepath(folder) {
-			settings.gamepath = folder;
-			settings.zip = path.join(settings.gamepath + "/northstar.zip");
-			saveSettings();
-			win.webContents.send("newpath", settings.gamepath);
-			ipcMain.emit("newpath", null, settings.gamepath);
-		}
+		if (! forcedialog) {
+			function setGamepath(folder, forcedialog) {
+				settings.gamepath = folder;
+				settings.zip = path.join(settings.gamepath + "/northstar.zip");
+				saveSettings();
+				win.webContents.send("newpath", settings.gamepath);
+				ipcMain.emit("newpath", null, settings.gamepath);
+			}
 
-		let gamepath = await findgame();
-		if (gamepath) {
-			setGamepath(gamepath);
-			return;
-		}
+			let gamepath = await findgame();
+			if (gamepath) {
+				setGamepath(gamepath);
+				return;
+			}
 
-		alert(lang("general.missingpath"));
+			winAlert(lang("general.missingpath"));
+		}
 
 		// Fallback to manual selection
 		dialog.showOpenDialog({properties: ["openDirectory"]}).then(res => {
-- 
cgit v1.2.3