Merge pull request #6 from de-luca/feature/readFile-instead-of-require

de-luca · web-flow · commit f35eab13968c · 2016-07-12T21:49:37.000+02:00
Use readFileSync instead of require
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "electron-json-config",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "Simply set and get configuration from a json file for your Electron app",
   "main": "src/index.js",
   "keywords": [
diff --git a/src/index.js b/src/index.js
@@ -8,7 +8,7 @@ const file = (electron.app || electron.remote.app).getPath('userData')+'/config.
 if(!exists.sync(file)) {
   fs.writeFileSync(file, '{}');
 }
-var config = require(file);
+var config = JSON.parse(fs.readFileSync(file));
 
 const sync = function() {
   fs.writeFileSync(file, JSON.stringify(config));

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "electron-json-config",`
`3`		`- "version": "1.1.0",`
	`3`	`+ "version": "1.2.0",`
`4`	`4`	`"description": "Simply set and get configuration from a json file for your Electron app",`
`5`	`5`	`"main": "src/index.js",`
`6`	`6`	`"keywords": [`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const file = (electron.app \|\| electron.remote.app).getPath('userData')+'/config.`
`8`	`8`	`if(!exists.sync(file)) {`
`9`	`9`	`fs.writeFileSync(file, '{}');`
`10`	`10`	`}`
`11`		`-var config = require(file);`
	`11`	`+var config = JSON.parse(fs.readFileSync(file));`
`12`	`12`
`13`	`13`	`const sync = function() {`
`14`	`14`	`fs.writeFileSync(file, JSON.stringify(config));`