diff options
author | Lon Ingram <lawnsea@gmail.com> | 2016-08-16 15:47:33 -0500 |
---|---|---|
committer | Lon Ingram <lawnsea@gmail.com> | 2016-08-16 18:23:53 -0500 |
commit | bbe0a94d6e5e909361d3f1195b1bc72dc31e233f (patch) | |
tree | 41aaa7afc8ba3c8e103e42dd84582b9dd7b7b690 | |
parent | a6a0e50551ff7a7f7349ea67d35dcda04ce26cfd (diff) | |
download | handlebars.js-bbe0a94d6e5e909361d3f1195b1bc72dc31e233f.zip handlebars.js-bbe0a94d6e5e909361d3f1195b1bc72dc31e233f.tar.gz handlebars.js-bbe0a94d6e5e909361d3f1195b1bc72dc31e233f.tar.bz2 |
Walk up data frames for nested @partial-block
The root cause of #1218 is that `invokePartial` creates a stack of data frames
for nested partial blocks, but `resolvePartial` always uses the value at top of
the stack without "popping" it. The result is an infinite recursive loop, as
references to `@partial-block` in the partial at the top of the stack resolve to
itself.
So, walk up the stack of data frames when evaluating. This is accomplished by
1) setting the `partial-block` property to `noop` after use and
2) using `_parent['partial-block']` if `partial-block` is `noop`
Fix #1218
-rw-r--r-- | lib/handlebars/runtime.js | 7 | ||||
-rw-r--r-- | spec/partials.js | 14 |
2 files changed, 20 insertions, 1 deletions
diff --git a/lib/handlebars/runtime.js b/lib/handlebars/runtime.js index 55eb1c1..7426f1f 100644 --- a/lib/handlebars/runtime.js +++ b/lib/handlebars/runtime.js @@ -197,7 +197,12 @@ export function wrapProgram(container, i, fn, data, declaredBlockParams, blockPa export function resolvePartial(partial, context, options) { if (!partial) { if (options.name === '@partial-block') { - partial = options.data['partial-block']; + let data = options.data; + while (data['partial-block'] === noop) { + data = data._parent; + } + partial = data['partial-block']; + data['partial-block'] = noop; } else { partial = options.partials[options.name]; } diff --git a/spec/partials.js b/spec/partials.js index d3ead74..07d1c0d 100644 --- a/spec/partials.js +++ b/spec/partials.js @@ -270,6 +270,20 @@ describe('partials', function() { true, 'success'); }); + it('should render nested partial blocks', function() { + shouldCompileToWithPartials( + '.template-start.{{#> outer}}{{value}}{{/outer}}.template-end.', + [ + {value: 'success'}, + {}, + { + outer: '.outer-start.{{#> nested}}.outer-partial-block-start.{{> @partial-block}}.outer-partial-block-end.{{/nested}}.outer-end.', + nested: '.nested-start.{{> @partial-block}}.nested-end.' + } + ], + true, + '.template-start..outer-start..nested-start..outer-partial-block-start.success.outer-partial-block-end..nested-end..outer-end..template-end.'); + }); }); describe('inline partials', function() { |