Reuse root path variable
authorKevin Sawicki <kevinsawicki@gmail.com>
Thu, 23 Jun 2016 22:56:29 +0000 (15:56 -0700)
committerKevin Sawicki <kevinsawicki@gmail.com>
Thu, 23 Jun 2016 22:56:29 +0000 (15:56 -0700)
lib/common/reset-search-paths.js

index 5cec77b..924cb38 100644 (file)
@@ -9,21 +9,19 @@ module.paths = []
 module.parent.paths = []
 
 // Prevent Node from adding paths outside this app to search paths.
+const resourcesPathWithTrailingSlash = process.resourcesPath + path.sep
 const originalNodeModulePaths = Module._nodeModulePaths
 Module._nodeModulePaths = function (from) {
   const paths = originalNodeModulePaths(from)
-
-  // If "from" is outside the app then we do nothing.
-  const rootPath = process.resourcesPath + path.sep
   const fromPath = path.resolve(from) + path.sep
-  const skipOutsidePaths = fromPath.startsWith(rootPath)
-  if (skipOutsidePaths) {
+  // If "from" is outside the app then we do nothing.
+  if (fromPath.startsWith(resourcesPathWithTrailingSlash)) {
     return paths.filter(function (candidate) {
-      return candidate.startsWith(rootPath)
+      return candidate.startsWith(resourcesPathWithTrailingSlash)
     })
+  } else {
+    return paths
   }
-
-  return paths
 }
 
 // Patch Module._resolveFilename to always require the Electron API when