Merge pull request #36 from jdelStrother/extensions

gdiggs · gdiggs · commit 2cc529779909 · 2015-10-16T08:49:27.000-04:00
Fix “Cannot read property 'indexOf' of undefined” in exclusionBasedFileListBuilder
diff --git a/bin/eslint.js b/bin/eslint.js
@@ -80,7 +80,7 @@ function exclusionBasedFileListBuilder(excludePaths) {
     allFiles.forEach(function(file, i, a){
       if(excludePaths.indexOf(file.split("/code/")[1]) < 0) {
         if(fs.lstatSync(file).isFile()) {
-          if (!isFileIgnoredByLibrary(file) && isFileWithMatchingExtension(file)) {
+          if (!isFileIgnoredByLibrary(file) && isFileWithMatchingExtension(file, extensions)) {
             analysisFiles.push(file);
           }
         }

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ function exclusionBasedFileListBuilder(excludePaths) {`
`80`	`80`	`allFiles.forEach(function(file, i, a){`
`81`	`81`	`if(excludePaths.indexOf(file.split("/code/")[1]) < 0) {`
`82`	`82`	`if(fs.lstatSync(file).isFile()) {`
`83`		`- if (!isFileIgnoredByLibrary(file) && isFileWithMatchingExtension(file)) {`
	`83`	`+ if (!isFileIgnoredByLibrary(file) && isFileWithMatchingExtension(file, extensions)) {`
`84`	`84`	`analysisFiles.push(file);`
`85`	`85`	`}`
`86`	`86`	`}`