Merge pull request #149 from carrot/refactors

Refactors

lib/api/add.js

@@ -8,8 +8,8 @@ import Template from '../template'
  * @return {Promise} - Promise for Template instance.
  */
 export default function (sprout, name, src) {
-  const template = new Template(sprout, name, src)
-  return template.save().then((template) => {
+  const template = new Template({ sprout: sprout, name: name, src: src })
+  return template.save().then(function (template) {
     sprout.templates[name] = template
   })
 }

lib/api/remove.js

@@ -9,9 +9,7 @@ import W from 'when'
 export default function (sprout, name) {
   const template = sprout.templates[name]
   if (template) {
-    return template.remove().then(() => {
-      delete sprout.templates[name]
-    })
+    return template.remove().then((_) => delete sprout.templates[name])
   }
   return W.reject(new Error(`template ${name} does not exist`))
 }

lib/index.js

@@ -9,32 +9,27 @@ import api from './api'
  * @param {String} basePath - Path to directory containing Sprout templates.
  * @return {Function} - Sprout instance.
  */
-class Sprout {
+class Sprout extends EventEmitter {
 
   constructor (basePath) {
+    super()
     this.templates = {}
     this.path = basePath
-    this.emitter = new EventEmitter()
 
-    var lstat, dirs, name, dir
-    if (!fs.existsSync(this.path)) {
-      throw new Error(this.path + ' does not exist')
-    } else {
-      lstat = fs.lstatSync(this.path)
-      if (!lstat.isDirectory()) {
-        throw new Error(this.path + ' is not a directory')
-      }
+    try { fs.accessSync(this.path) } catch (err) {
+      throw new Error(`${this.path} does not exist`)
+    }
+
+    if (!fs.lstatSync(this.path).isDirectory()) {
+      throw new Error(`${this.path} is not a directory`)
     }
 
-    dirs = fs.readdirSync(this.path)
-    for (var i = 0; i < dirs.length; i++) {
-      name = dirs[i]
-      dir = path.join(this.path, name)
-      lstat = fs.lstatSync(dir)
+    fs.readdirSync(this.path).map((name) => {
+      const lstat = fs.lstatSync(path.join(this.path, name))
       if (lstat.isDirectory() || lstat.isSymbolicLink()) {
-        this.templates[name] = new Template(this, name)
+        this.templates[name] = new Template({ sprout: this, name: name })
       }
-    }
+    })
   }
 
   /*