diff --git a/lib/handlebars/compiler/ast.js b/lib/handlebars/compiler/ast.js index e05ceec63..05de762b6 100644 --- a/lib/handlebars/compiler/ast.js +++ b/lib/handlebars/compiler/ast.js @@ -79,11 +79,11 @@ var AST = { this.strip.inlineStandalone = true; }, - BlockNode: function(mustache, program, inverse, strip, locInfo) { + BlockNode: function(sexpr, program, inverse, strip, locInfo) { LocationInfo.call(this, locInfo); this.type = 'block'; - this.mustache = mustache; + this.sexpr = sexpr; this.program = program; this.inverse = inverse; this.strip = strip; @@ -93,17 +93,11 @@ var AST = { } }, - RawBlockNode: function(mustache, content, close, locInfo) { + RawBlockNode: function(sexpr, content, locInfo) { LocationInfo.call(this, locInfo); - if (mustache.sexpr.id.original !== close) { - throw new Exception(mustache.sexpr.id.original + " doesn't match " + close, this); - } - - content = new AST.ContentNode(content, locInfo); - this.type = 'block'; - this.mustache = mustache; + this.sexpr = sexpr; this.program = new AST.ProgramNode([content], {}, locInfo); }, diff --git a/lib/handlebars/compiler/compiler.js b/lib/handlebars/compiler/compiler.js index 1303d8fc4..51e66203a 100644 --- a/lib/handlebars/compiler/compiler.js +++ b/lib/handlebars/compiler/compiler.js @@ -108,7 +108,7 @@ Compiler.prototype = { }, block: function(block) { - var mustache = block.mustache, + var sexpr = block.sexpr, program = block.program, inverse = block.inverse; @@ -120,7 +120,6 @@ Compiler.prototype = { inverse = this.compileProgram(inverse); } - var sexpr = mustache.sexpr; var type = this.classifySexpr(sexpr); if (type === "helper") { diff --git a/lib/handlebars/compiler/helpers.js b/lib/handlebars/compiler/helpers.js index 20f13d656..c9a35c24d 100644 --- a/lib/handlebars/compiler/helpers.js +++ b/lib/handlebars/compiler/helpers.js @@ -12,17 +12,35 @@ export function stripComment(comment) { .replace(/-?-?~?\}\}$/, ''); } +export function prepareRawBlock(openRawBlock, content, close, locInfo) { + /*jshint -W040 */ + if (openRawBlock.sexpr.id.original !== close) { + var errorNode = { + firstLine: openRawBlock.sexpr.firstLine, + firstColumn: openRawBlock.sexpr.firstColumn + }; + + throw new Exception(openRawBlock.sexpr.id.original + " doesn't match " + close, errorNode); + } -export function prepareBlock(mustache, program, inverseAndProgram, close, inverted, locInfo) { + return new this.RawBlockNode(openRawBlock.sexpr, content, locInfo); +} + +export function prepareBlock(openBlock, program, inverseAndProgram, close, inverted, locInfo) { /*jshint -W040 */ - if (mustache.sexpr.id.original !== close.path.original) { - throw new Exception(mustache.sexpr.id.original + ' doesn\'t match ' + close.path.original, mustache); + if (openBlock.sexpr.id.original !== close.path.original) { + var errorNode = { + firstLine: openBlock.sexpr.firstLine, + firstColumn: openBlock.sexpr.firstColumn + }; + + throw new Exception(openBlock.sexpr.id.original + ' doesn\'t match ' + close.path.original, errorNode); } var inverse = inverseAndProgram && inverseAndProgram.program; var strip = { - left: mustache.strip.left, + left: openBlock.strip.left, right: close.strip.right, // Determine the standalone candiacy. Basically flag our content as being possibly standalone @@ -31,7 +49,7 @@ export function prepareBlock(mustache, program, inverseAndProgram, close, invert closeStandalone: isPrevWhitespace((inverse || program).statements) }; - if (mustache.strip.right) { + if (openBlock.strip.right) { omitRight(program.statements, null, true); } @@ -62,9 +80,9 @@ export function prepareBlock(mustache, program, inverseAndProgram, close, invert } if (inverted) { - return new this.BlockNode(mustache, inverse, program, strip, locInfo); + return new this.BlockNode(openBlock.sexpr, inverse, program, strip, locInfo); } else { - return new this.BlockNode(mustache, program, inverse, strip, locInfo); + return new this.BlockNode(openBlock.sexpr, program, inverse, strip, locInfo); } } diff --git a/lib/handlebars/compiler/printer.js b/lib/handlebars/compiler/printer.js index 7654245c5..48532938c 100644 --- a/lib/handlebars/compiler/printer.js +++ b/lib/handlebars/compiler/printer.js @@ -40,7 +40,7 @@ PrintVisitor.prototype.block = function(block) { out = out + this.pad("BLOCK:"); this.padding++; - out = out + this.accept(block.mustache); + out = out + this.accept(block.sexpr); if (block.program) { out = out + this.pad("PROGRAM:"); this.padding++; diff --git a/spec/ast.js b/spec/ast.js index 7118c72dd..9b3edb03b 100644 --- a/spec/ast.js +++ b/spec/ast.js @@ -71,7 +71,7 @@ describe('ast', function() { it('should throw on mustache mismatch', function() { shouldThrow(function() { handlebarsEnv.parse("\n {{#foo}}{{/bar}}"); - }, Handlebars.Exception, "foo doesn't match bar - 2:2"); + }, Handlebars.Exception, "foo doesn't match bar - 2:5"); }); it('stores location info', function(){ diff --git a/src/handlebars.yy b/src/handlebars.yy index ccc0d2248..233088ba3 100644 --- a/src/handlebars.yy +++ b/src/handlebars.yy @@ -17,16 +17,20 @@ statement | block -> $1 | rawBlock -> $1 | partial -> $1 - | CONTENT -> new yy.ContentNode($1, @$) + | content -> $1 | COMMENT -> new yy.CommentNode(yy.stripComment($1), yy.stripFlags($1, $1), @$) ; +content + : CONTENT -> new yy.ContentNode($1, @$) + ; + rawBlock - : openRawBlock CONTENT END_RAW_BLOCK -> new yy.RawBlockNode($1, $2, $3, @$) + : openRawBlock content END_RAW_BLOCK -> yy.prepareRawBlock($1, $2, $3, @$) ; openRawBlock - : OPEN_RAW_BLOCK sexpr CLOSE_RAW_BLOCK -> new yy.MustacheNode($2, null, '', '', @$) + : OPEN_RAW_BLOCK sexpr CLOSE_RAW_BLOCK -> { sexpr: $2 } ; block @@ -35,11 +39,11 @@ block ; openBlock - : OPEN_BLOCK sexpr CLOSE -> new yy.MustacheNode($2, null, $1, yy.stripFlags($1, $3), @$) + : OPEN_BLOCK sexpr CLOSE -> { sexpr: $2, strip: yy.stripFlags($1, $3) } ; openInverse - : OPEN_INVERSE sexpr CLOSE -> new yy.MustacheNode($2, null, $1, yy.stripFlags($1, $3), @$) + : OPEN_INVERSE sexpr CLOSE -> { sexpr: $2, strip: yy.stripFlags($1, $3) } ; inverseAndProgram