runtime: lazily set mods

bem · Apr 27, 2015 · 8b8ab29 · 8b8ab29
1 parent 3f9e352
commit 8b8ab29
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/bemhtml/runtime/index.js b/lib/bemhtml/runtime/index.js
@@ -199,7 +199,9 @@ BEMHTML.prototype.runOne = function runOne(json) {
   context.ctx = json;
   if (json.block) {
     context.block = json.block;
-    context.mods = json.mods;
+
+    // TODO(indutny): this might be a performance hit, consider it
+    context.mods = json.mods || {};
   } else if (!json.elem) {
     context.block = '';
   } else if (oldCurrBlock) {

diff --git a/test/runtime-test.js b/test/runtime-test.js
@@ -74,6 +74,14 @@ describe('BEMHTML compiler/Runtime', function() {
     }, { tag: false, content: 'ok' }, 'ok');
   });
 
+  it('should lazily define mods', function() {
+    test(function() {
+      block('b1').content()(function() {
+        return this.mods.a || 'yes';
+      });
+    }, { block: 'b1' }, '<div class="b1">yes</div>');
+  });
+
   describe('mix', function() {
     it('should avoid loops', function() {
       test(function() {