Setup config location as a ES6 class w/ static getters

Author: jakejarrett

app/public/js/common/configLocation.js

@@ -0,0 +1,53 @@
+const fs = require('fs-extra');
+const userHome = require('user-home');
+
+class Configuration {
+
+    /**
+     * Get the configuration path
+     * 
+     * @returns {string} The location of the config
+     */
+    static get path () {
+        let userConfigPath = null;
+
+        /** Linux platforms - XDG Standard */
+        if (process.platform === 'win32') {
+            userConfigPath = `${userHome}/.config/Soundnode/userConfig.json`;
+        }
+
+
+        /** Linux platforms - XDG Standard */
+        if (process.platform === 'linux') {
+            userConfigPath = `${userHome}/.config/Soundnode/userConfig.json`;
+        }
+
+        /** Mac os configuration location */
+        if (process.platform === 'darwin') {
+            userConfigPath = `${userHome}/Library/Preferences/Soundnode/userConfig.json`;
+        }
+
+        return userConfigPath;
+    }
+
+    /**
+     * Get the config file
+     * 
+     * @returns {Object} Parsed version of the saved file
+     */
+    static get file () {
+        return JSON.parse(fs.readFileSync(`${this.path}`, 'utf-8'));
+    }
+
+    /**
+     * Ensure the config exists
+     * 
+     * @returns {Boolean} True if the file exists
+     */
+    static get configExists () {
+        return fs.existsSync(this.path);
+    }
+
+}
+
+module.exports = Configuration;

app/public/js/system/guiConfig.js

@@ -5,6 +5,7 @@ const {
 } = require('electron');
 const fs = require('fs-extra');
 const userHome = require('user-home');
+const Configuration = require('../common/configLocation');
 
 let guiConfig = {};
 
@@ -29,19 +30,7 @@ guiConfig.maximize = function () {
 };
 
 guiConfig.logOut = function () {
-  let _userConfigPath = `${__dirname}/app/public/js/system/userConfig.json`; // Windows specific for now
-
-  /** Linux platforms - XDG Standard */
-  if (process.platform === 'linux') {
-    _userConfigPath = `${userHome}/.config/Soundnode/userConfig.json`;
-  }
-
-  /** Mac os configuration location */
-  if (process.platform === 'darwin') {
-    _userConfigPath = `${userHome}/Library/Preferences/Soundnode/userConfig.json`;
-  }
-
-  fs.removeSync(`${_userConfigPath}`);
+  fs.removeSync(Configuration.path);
   this.destroy();
 };
 

app/public/js/system/settings.js

@@ -2,19 +2,8 @@
 
 const ua = require('universal-analytics');
 const fs = require('fs');
-const userHome = require('user-home');
-
-let _userConfigPath = `${__dirname}/app/public/js/system/userConfig.json`; // Windows specific for now
-/** Linux platforms - XDG Standard */
-if (process.platform === 'linux') {
-    _userConfigPath = `${userHome}/.config/Soundnode/userConfig.json`;
-}
-
-/** Mac os configuration location */
-if (process.platform === 'darwin') {
-    _userConfigPath = `${userHome}/Library/Preferences/Soundnode/userConfig.json`;
-}
-const userConfig = JSON.parse(fs.readFileSync(`${_userConfigPath}`, 'utf-8'));
+const Configuration = require('../common/configLocation');
+const userConfig = Configuration.file;
 
 // Set up some core settings
 window.settings = {};

main.js

@@ -9,25 +9,12 @@ const {
   Menu
 } = require('electron');
 const windowStateKeeper = require('electron-window-state');
-const userHome = require('user-home');
+const Configuration = require('./app/public/js/common/configLocation');
 
 // custom constants
 const clientId = '342b8a7af638944906dcdb46f9d56d98';
 const redirectUri = 'http://sc-redirect.herokuapp.com/callback.html';
 const SCconnect = `https://soundcloud.com/connect?&client_id=${clientId}&redirect_uri=${redirectUri}&response_type=token`;
-let _userConfigPath = `${__dirname}/app/public/js/system/userConfig.json`; // Windows specific for now
-
-/** Linux platforms - XDG Standard */
-if (process.platform === 'linux') {
-  _userConfigPath = `${userHome}/.config/Soundnode/userConfig.json`;
-}
-
-/** Mac os configuration location */
-if (process.platform === 'darwin') {
-  _userConfigPath = `${userHome}/Library/Preferences/Soundnode/userConfig.json`;
-}
-
-const userConfigPath = _userConfigPath.slice(0); // Just to make sure it doesn't keep reference to the mutable string.
 
 let mainWindow;
 let authenticationWindow;
@@ -37,7 +24,7 @@ app.on('ready', () => {
 });
 
 function checkUserConfig() {
-  const userConfigExists = fs.existsSync(userConfigPath);
+  const userConfigExists = Configuration.configExists;
 
   if (userConfigExists) {
     initMainWindow();
@@ -84,7 +71,7 @@ function authenticateUser() {
 }
 
 function setUserData(accessToken) {
-  fs.writeFileSync(userConfigPath, JSON.stringify({
+  fs.writeFileSync(Configuration.path, JSON.stringify({
     accessToken: accessToken,
     clientId: clientId
   }), 'utf-8');