refactor: client store code style (#3683)

nuxt · Sep 23, 2018 · 054ea79 · 054ea79
1 parent bc1cecf
commit 054ea79
Showing 1 changed file with 8 additions and 11 deletions.
diff --git a/lib/app/store.js b/lib/app/store.js
@@ -3,20 +3,15 @@ import Vuex from 'vuex'
 
 Vue.use(Vuex)
 
-// Recursive find files in {srcDir}/{dir.store}
 const files = require.context('@/<%= dir.store %>', true, /^\.\/(?!<%= ignorePrefix %>)[^.]+\.(<%= extensions %>)$/)
 const filenames = files.keys()
 
 // Store
 let storeData = {}
 
 // Check if {dir.store}/index.js exists
-let indexFilename
-filenames.forEach((filename) => {
-  if (filename.indexOf('./index.') !== -1) {
-    indexFilename = filename
-  }
-})
+const indexFilename = filenames.find(name => name.includes('./index.'))
+
 if (indexFilename) {
   storeData = getModule(indexFilename)
 }
@@ -35,21 +30,21 @@ if (typeof storeData !== 'function') {
     const namePath = name.split(/\//)
 
     name = namePath[namePath.length - 1]
-    if (name === 'state' || name === 'getters' || name === 'actions' || name === 'mutations') {
+    if (['state', 'getters', 'actions', 'mutations'].includes(name)) {
       const module = getModuleNamespace(storeData, namePath, true)
       appendModule(module, filename, name)
       continue
     }
 
-    // if file is foo/index.js
-    // it should save as foo
+    // If file is foo/index.js, it should be saved as foo
     const isIndex = (name === 'index')
     if (isIndex) {
       namePath.pop()
     }
 
     const module = getModuleNamespace(storeData, namePath)
     const fileModule = getModule(filename)
+
     name = namePath.pop()
     module[name] = module[name] || {}
 
@@ -100,7 +95,9 @@ function getModule (filename) {
 
 function getModuleNamespace (storeData, namePath, forAppend = false) {
   if (namePath.length === 1) {
-    if (forAppend) { return storeData }
+    if (forAppend) {
+      return storeData
+    }
     return storeData.modules
   }
   const namespace = namePath.shift()