Skip to content

Commit

Permalink
Merge pull request #1121 from shinypb/undefined_partial_error_message
Browse files Browse the repository at this point in the history
Include partial name in 'undefined partial' exception message
  • Loading branch information
kpdecker committed Oct 19, 2015
2 parents 9365b82 + 33b53ef commit d20f02d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/handlebars/base.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ HandlebarsEnvironment.prototype = {
extend(this.partials, name);
} else {
if (typeof partial === 'undefined') {
throw new Exception('Attempting to register a partial as undefined');
throw new Exception(`Attempting to register a partial called "${name}" as undefined`);
}
this.partials[name] = partial;
}
Expand Down
2 changes: 1 addition & 1 deletion spec/partials.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ describe('partials', function() {
shouldThrow(function() {
var undef;
handlebarsEnv.registerPartial('undefined_test', undef);
}, Handlebars.Exception, 'Attempting to register a partial as undefined');
}, Handlebars.Exception, 'Attempting to register a partial called "undefined_test" as undefined');
});

it('rendering template partial in vm mode throws an exception', function() {
Expand Down

0 comments on commit d20f02d

Please sign in to comment.