use VariableDeclarator instead of looping VariableDeclaration

import-js · Oct 26, 2018 · 8fe7df2 · 8fe7df2
1 parent 1834f05
commit 8fe7df2
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/src/rules/named.js b/src/rules/named.js
@@ -64,12 +64,9 @@ module.exports = {
       })
     }
 
-    function checkRequire(declarationNode) {
-      if(!options.commonjs) {
-        return
-      }
+    function checkRequire(node) {
+        if(!options.commonjs) return
 
-      declarationNode.declarations.forEach(function(node) {
         if(node.type !== 'VariableDeclarator') return
 
         if(!node.id || node.id.type !== 'ObjectPattern' || node.id.properties.length === 0) {
@@ -122,7 +119,6 @@ module.exports = {
             }
           }
         })
-      })
     }
 
     return {
@@ -136,7 +132,7 @@ module.exports = {
                                                     , 'ExportSpecifier'
                                                     ),
 
-      'VariableDeclaration': checkRequire,
+      'VariableDeclarator': checkRequire,
     }
 
   },