moved generating vendorPath to constructor

blossom · May 18, 2011 · c3e980d · c3e980d
1 parent 634c125
commit c3e980d
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/lib/compilers/stitch.js b/lib/compilers/stitch.js
diff --git a/src/compilers/stitch.coffee b/src/compilers/stitch.coffee
@@ -9,6 +9,10 @@ Compiler = require('./base').Compiler
 
 class exports.StitchCompiler extends Compiler
 
+  constructor: (options) ->
+    super options
+    @vendorPath = path.join(options.brunchPath, 'src/vendor')
+
   filePattern: ->
     [/\.coffee$/, /src\/.*\.js$/, new RegExp("#{@options.templateExtension}$")]
 
@@ -39,14 +43,11 @@ class exports.StitchCompiler extends Compiler
   # generate list of dependencies and preserve order of brunch libaries
   # like defined in options.dependencies
   collectDependencies: ->
-    filenames = fs.readdirSync @vendorPath()
-    filenames = helpers.filterFiles filenames, @vendorPath()
+    filenames = fs.readdirSync @vendorPath
+    filenames = helpers.filterFiles filenames, @vendorPath
 
     args = @options.dependencies.slice()
     args.unshift filenames
     additionalLibaries = _.without.apply @, args
     dependencies = @options.dependencies.concat additionalLibaries
-    _.map dependencies, (filename) => path.join(@vendorPath(), filename)
-
-  vendorPath: ->
-    @_vendor_path ||= path.join(@options.brunchPath, 'src/vendor')
+    _.map dependencies, (filename) => path.join(@vendorPath, filename)
diff --git a/test/collectDependencies.coffee b/test/collectDependencies.coffee
@@ -39,7 +39,7 @@ module.exports = testCase(
     ]
 
     compiler = new StitchCompiler options
-    compiler._vendor_path = 'test/fixtures/alternate_vendor'
+    compiler.vendorPath = 'test/fixtures/alternate_vendor'
 
     dependencyPaths = compiler.collectDependencies()
     test.deepEqual dependencyPaths, [