body: throw meaningful error

fix bem/bem-xjst#10
veged · Apr 1, 2014 · 3a40d0e · 3a40d0e
1 parent 2c0d30d
commit 3a40d0e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/xjst/compiler/entities/body.js b/lib/xjst/compiler/entities/body.js
@@ -250,7 +250,9 @@ Body.prototype.rollOutLocal = function rollOutLocal(ast, changes, body) {
   changes.forEach(function(change) {
     if (change.type === 'Literal')
       return;
-    assert.equal(change.type, 'ObjectExpression');
+    assert.equal(change.type,
+                 'ObjectExpression',
+                 'apply() and local() accepts only object literals');
     change.properties.forEach(function(property) {
       var keys = (property.key.name || property.key.value).split('.');
       var isGlobal = keys[0] === '$$global';

diff --git a/lib/xjst/utils.js b/lib/xjst/utils.js
@@ -52,6 +52,9 @@ utils.run = function run(templates, context) {
       if (change === null)
         return;
 
+      if (typeof change !== 'object')
+        throw new Error('apply() and local() accepts only object literals');
+
       Object.keys(change).forEach(function(key) {
         var parts = key.split('.'),
             newValue = change[key],