Fix rendering of paths that resolve to zero

Fixes #820
handlebars-lang · Jul 12, 2014 · 1fb7b51 · 1fb7b51
1 parent c90cfe2
commit 1fb7b51
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/handlebars/compiler/javascript-compiler.js b/lib/handlebars/compiler/javascript-compiler.js
@@ -369,7 +369,7 @@ JavaScriptCompiler.prototype = {
           return ' != null ? ' + lookup + ' : ' + current;
         } else {
           // Otherwise we can use generic falsy handling
-          return ' && ' + lookup;
+          return ' != null && ' + lookup;
         }
       }, true);
     }

diff --git a/spec/regressions.js b/spec/regressions.js
@@ -122,6 +122,10 @@ describe('Regressions', function() {
     shouldCompileTo('{{#foo}} This is {{bar}} ~ {{/foo}}', {foo: 0, bar: 'OK'}, ' This is  ~ ');
   });
 
+  it('GH-820: zero pathed rendering', function() {
+    shouldCompileTo('{{foo.bar}}', {foo: 0}, '');
+  });
+
   if (Handlebars.AST) {
     it("can pass through an already-compiled AST via compile/precompile", function() {
       equal(Handlebars.compile(new Handlebars.AST.ProgramNode([ new Handlebars.AST.ContentNode("Hello")]))(), 'Hello');