Wrong context when iterating an array with `undefined` inside a conditional block helper #1762

aleen42 · 2021-06-29T03:00:38Z

Recently I found that I have missed my context with the following snippet:

const Handlebars = require('handlebars');

Handlebars.compile('{{#each arr}}'
                   + '{{#if 1}}{{@index}}: {{../foo}}|{{/if}}'
                   + '{{/each}}')({
    arr : [1, 2, void 0, 4],
    foo : 'foo',
}); // => "0: foo|1: foo|2: |3: foo|"

Handlebars.compile('{{#each arr}}'
                   + '{{#if 0}}{{^}}{{@index}}: {{../foo}}|{{/if}}'
                   + '{{/each}}')({
    arr : [1, 2, void 0, 4],
    foo : 'foo',
}); // => "0: foo|1: foo|2: |3: foo|"

The problem only occurs inside a conditional block helper:

Handlebars.compile('{{#each arr}}'
                   + '{{@index}}: {{../foo}}|'
                   + '{{/each}}')({
    arr : [1, 2, void 0, 4],
    foo : 'foo',
}); // => "0: foo|1: foo|2: foo|3: foo|"

It is a common case that a conditional block helper should not create a new context, but actually, it has wrapped an empty context when iterating an array with undefined inside it.

The text was updated successfully, but these errors were encountered:

aleen42 changed the title ~~Wrong context when iterating an array with undefined inside a conditional block helper~~ Wrong context when iterating an array with undefined inside a conditional block helper Jun 29, 2021

jaylinski added bug needs investigation labels Nov 23, 2021

PitPik mentioned this issue Jun 19, 2022

Parent context wrong for first each iteration #1831

Open

Wrong context when iterating an array with undefined inside a conditional block helper #1762

Wrong context when iterating an array with undefined inside a conditional block helper #1762

Comments

aleen42 commented Jun 29, 2021 • edited

Wrong context when iterating an array with `undefined` inside a conditional block helper #1762

Wrong context when iterating an array with `undefined` inside a conditional block helper #1762

aleen42 commented Jun 29, 2021 •

edited