fix crash when no options are passed

karashiiro · Jan 23, 2020 · 429101f · 429101f
1 parent 61a8045
commit 429101f
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/node-machina-ffxiv.js b/node-machina-ffxiv.js
@@ -100,8 +100,6 @@ const MachinaFFXIV = (() => {
                     throw new TypeError("port must be a number.");
                 } else if (options.port) {
                     this[port] = options.port;
-                } else {
-                    this[port] = 13346;
                 }
 
                 if (options.hasWine && typeof options.hasWine !== 'boolean') {
@@ -114,11 +112,16 @@ const MachinaFFXIV = (() => {
                     throw new TypeError("winePrefix must be a string.");
                 } else if (options.winePrefix) {
                     this[winePrefix] = options.winePrefix;
-                } else {
-                    this[winePrefix] = "$HOME/.Wine";
                 }
             }
 
+            if (!this[port]) {
+                this[port] = 13346;
+            }
+            if (!this[winePrefix]) {
+                this[winePrefix] = "$HOME/.Wine";
+            }
+
             // Folders
             const remoteDatapath = (options && options.remoteDataPath) || path.join(__dirname, './remote-data');
             if (!fs.existsSync(remoteDatapath)) {