Merge pull request #1087 from denniskuczynski/each_object_empty_string_key

Fix #each when last object entry has empty key

2 files changed, 11 insertions, 1 deletions

diff --git a/lib/handlebars/helpers/each.js b/lib/handlebars/helpers/each.js
index d39a300..9b1629b 100644
--- a/lib/handlebars/helpers/each.js
+++ b/lib/handlebars/helpers/each.js
@@ -68,7 +68,7 @@ export default function(instance) {
             i++;
           }
         }
-        if (priorKey) {
+        if (priorKey !== undefined) {
           execIteration(priorKey, i - 1, true);
         }
       }
diff --git a/spec/builtins.js b/spec/builtins.js
index a7f6204..f06a1ad 100644
--- a/spec/builtins.js
+++ b/spec/builtins.js
@@ -225,6 +225,16 @@ describe('builtin helpers', function() {
                 'each with array function argument ignores the contents when empty');
     });
 
+    it('each object when last key is an empty string', function() {
+      var string = '{{#each goodbyes}}{{@index}}. {{text}}! {{/each}}cruel {{world}}!';
+      var hash = {goodbyes: {'a': {text: 'goodbye'}, b: {text: 'Goodbye'}, '': {text: 'GOODBYE'}}, world: 'world'};
+
+      var template = CompilerContext.compile(string);
+      var result = template(hash);
+
+      equal(result, '0. goodbye! 1. Goodbye! 2. GOODBYE! cruel world!', 'Empty string key is not skipped');
+    });
+
     it('data passed to helpers', function() {
       var string = '{{#each letters}}{{this}}{{detectDataInsideEach}}{{/each}}';
       var hash = {letters: ['a', 'b', 'c']};