Force {precompiled: false} option within rendering methods.

Fixes #2
ericf · Feb 1, 2013 · 9c1942a · 9c1942a
1 parent 49ddc17
commit 9c1942a
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/HISTORY.md b/HISTORY.md
@@ -1,6 +1,12 @@
 Express3 Handlebars Change History
 ==================================
 
+NEXT
+----
+
+* Force `{precompiled: false}` option within `render()` and `renderView()`
+  methods to prevent trying to render with precompiled templates.
+
 0.1.2 (2013-01-10)
 ------------------
 

diff --git a/lib/express3-handlebars.js b/lib/express3-handlebars.js
@@ -152,6 +152,12 @@ extend(ExpressHandlebars.prototype, {
             }
         }
 
+        // Force `{precompiled: false}` option, before passing options along to
+        // `getPartials()` and `getTemplate()` methods.
+        if (options.precompiled) {
+            options = extend({}, options, {precompiled: false});
+        }
+
         async.parallel({
             partials: this.getPartials.bind(this, options),
             template: this.getTemplate.bind(this, filePath, options)
@@ -188,6 +194,12 @@ extend(ExpressHandlebars.prototype, {
             layoutPath = path.join(this.layoutsDir, layoutPath);
         }
 
+        // Force `{precompiled: false}` option, before passing options along to
+        // `getPartials()` and `getTemplate()` methods.
+        if (options.precompiled) {
+            options = extend({}, options, {precompiled: false});
+        }
+
         async.waterfall([
             this.render.bind(this, viewPath, options),
             renderLayout.bind(this)