add comment why we have to set opts.view to null

dodo · Jun 7, 2012 · f38762b · f38762b
1 parent 1dbd185
commit f38762b
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/template.coffee b/src/template.coffee
@@ -31,10 +31,8 @@ class Template extends EventEmitter
     constructor: (opts = {}, template) ->
         # options
         [template, opts] = [opts, {}] if typeof opts is 'function'
-        view = null
+        view = opts.view
-        if typeof(opts.view) is not "undefined"
+        opts.view = null # prevent infinite recursion
-            view = opts.view
-            opts.view = null
         # defaults
         opts.encoding ?= 'utf-8'
         opts.doctype ?= off
@@ -57,7 +55,7 @@ class Template extends EventEmitter
         # instantiate
         @xml = new ExtendedBuilder opts
         @xml.template = this
-        @xml.view     = view 
+        @xml.view     = view
         # override query
         old_query = @xml.query
         @xml.query = (type, tag, key) ->