Support for Number mods values

[golyshevd]

see details in commit

The modifier serializes to class well, but template does not match

+ expected - actual

      -<h1 class="title title_level_2"></h1>
      +<h2 class="title title_level_2"></h2>

[golyshevd]

@remnev @truerenton

[qfox]

👍 Yeah, looks like it should.

[qfox]

https://github.com/bem/bh/blob/master/lib/bh.js#L822-L831

To fix that you need:

                     if (decl.elemMod) {
                         conds.push(
                             'json.elemMods && json.elemMods["' + decl.elemMod + '"] === ' +
-                                (decl.elemModVal === true || '"' + decl.elemModVal + '"'));
+                                (decl.elemModVal === true || Number(decl.elemModVal) || '"' + decl.elemModVal + 
                     }
                     if (decl.blockMod) {
                         conds.push(
                             'json.mods && json.mods["' + decl.blockMod + '"] === ' +
-                                (decl.blockModVal === true || '"' + decl.blockModVal + '"'));
+                                (decl.blockModVal === true || Number(decl.blockModVal) || '"' + decl.blockModVal + '"'));
                     }

Or something similar.

[remnev]

I'll be fixing, ok?

[mishanga]

Надо не декларацию к числу приводить, а значение модификатора в bemjson к строке:

'json.elemMods && String(json.elemMods["' + decl.elemMod + '"]) === ' +
    (decl.elemModVal === true || '"' + decl.elemModVal + '"'));

Так будет правильнее и быстрее по производительности.

[qfox]

String(true) === 'true' — не страшно? :-)

[golyshevd]

Почему так быстрее? у тебя новый вызов в рантайме, а у юры при генерации

[golyshevd]

а примерно понятно

UPD: ой нет )

[qfox]

А, стоп. У меня кривой код.

[remnev]

String(true) === 'true' — не страшно? :-)

страшно ведь, так и произойдет

[mishanga]

Да, про производительность наврал.

[mishanga]

В целом задача выглядит синтетической. В приведенном примере не может быть больше 7 значений модификатора, и их можно выразить через ctx.tag('h' + (ctx.mod('level') || 1)).
Использование численных модификаторов почти всегда говорит о неправильной архитектуре блока.

[golyshevd]

Если говорить о конкретном блоке, то я не захотел писать как ты, а написал 7 шаблонов для каждого модификатора чтобы не генерировать название тега и случайно на такое наткнулся. Лично мне показалось абсолютно безобидным то, что значение модификатора является числом