src/modules/settings.js now provides a function

This fixes a couple issues where the main process wouldn't actually get
changes made to the settings, this fixes that.

On top of this, changing settings is now done with `settings.set()`

There shouldn't be any breakage from this change, but I suppose it is
possible. Especially because the `settings()` function still does
contain backup options set, meaning `settings.nsargs` is technically
still valid, but dont expect it to actually be updated when that
variable is changed, its merely here to avoid any problems.

1 files changed, 6 insertions, 6 deletions

diff --git a/src/index.js b/src/index.js
index 74fe669..82805d1 100644
--- a/src/index.js
+++ b/src/index.js
@@ -74,7 +74,7 @@ function start() {
 	}; send = win.send;
 
 	ipcMain.on("exit", () => {
-		if (settings.originkill) {
+		if (settings().originkill) {
 			is_running.origin().then((running) => {
 				if (running) {
 					kill.origin().then(process.exit(0))
@@ -111,7 +111,7 @@ function start() {
 	ipcMain.on("delete-install-cache", () => {
 		let delete_dirs = [
 			path.join(app.getPath("cache"), "vipertmp"),
-			path.join(settings.gamepath, "northstar.zip")
+			path.join(settings().gamepath, "northstar.zip")
 		]
 
 		for (let i = 0; i < delete_dirs.length; i++) {
@@ -197,7 +197,7 @@ function start() {
 	});
 
 	ipcMain.on("save-settings", (event, obj) => {
-		settings.save(obj, false)
+		settings().save(obj, false);
 	});
 
 	ipcMain.on("saved-settings", (event, obj) => {
@@ -219,7 +219,7 @@ function start() {
 	})
 
 	// start auto-update process
-	if (settings.autoupdate) {
+	if (settings().autoupdate) {
 		if (cli.hasParam("no-vp-updates")) {
 			update.northstar_autoupdate();
 		} else {
@@ -265,8 +265,8 @@ ipcMain.on("launch-ns", () => {launch()});
 ipcMain.on("launch-vanilla", () => {launch("vanilla")});
 
 ipcMain.on("setlang", (event, lang) => {
-	settings.lang = lang;
-	settings.save();
+	settings().set("lang", lang);
+	settings().save();
 });
 
 ipcMain.on("update-northstar", async (e, force_install) => {