Не применяется BEMHTML-шаблон

Спасибо форуму, сборка происходит нормально. Но результатов BEMHTML на клиенте пока нет.

В JavaScript код такой:

        var reposBEMJSON = this.repos.map(function(repo) {

            return {
                block: 'test',
                //elem: 'repo',
                content: '1111',
                info: {
                    name: repo.name,
                    url: repo.html_url
                }
            }
        });

        var html = BEMHTML.apply(reposBEMJSON)
        console.log(html);

https://github.com/varya/varya.github.com/blob/features/github-repos/desktop.blocks/github/github.js#L49

Зависимости по технологиям указываю: https://github.com/varya/varya.github.com/blob/features/github-repos/desktop.blocks/github/github.deps.js#L12

Судя по коду index.min.js шаблон всё-таки включен в сборку:

if(!__$ctx.elem&&__$ctx.block==="test"&&__$ctx._mode==="content"){__$ctx.__$a=0;return"XXXX"}

Но в консоль выводится undefined. В браузер приезжает свежий, незакешированной index.min.js — я проверила.

В чём может быть проблема и как дебажить дальше?

10 years ago

tadatuta

@varya не хватает зависимости от блока i-bem в технологии bemhtml

10 years ago

varya

Хм, не очень понятно.

Я добавила зависимость к блоку test:

[{

    tech: 'bemhtml',
    mustDeps: [
      { block: 'i-bem' }
    ]

}]

Но это не помогло. Помогла только зависимость для блока github (для которого и пишу JavaScript):

{

    tech: 'js',
    mustDeps: [
      { tech: 'bemhtml', block: 'test' }
    ]

}, {
    tech: 'js',
    mustDeps: { tech: 'bemhtml', block: 'i-bem' }

}]

Как-то нелогично. Почему первый вариант не подходит?

10 years ago

varya

А можно ещё раз, что от чего должно зависеть? Я указала i-bem https://github.com/varya/varya.github.com/blob/features/github-repos/desktop.blocks/github/github.deps.js#L12 Теперь я получаю не undefined, но всё равно результат дефолтных шаблонов:

<div class="test">1111</div><div class="test">1111</div>

А хочу, чтобы применился шаблон на блок test: https://github.com/varya/varya.github.com/blob/features/github-repos/desktop.blocks/test/test.bemhtml

10 years ago

tadatuta

@varya прислал PR: https://github.com/varya/varya.github.com/pull/63/files с пояснениями по диффу

10 years ago

varya

Спасибо, всё заработало! А дальше уже переделать на зависимость от собственного элемента оказалось несложно https://github.com/varya/varya.github.com/commit/410b359d0dc7f9abc8bbe321439a304651a9d2bd

10 years ago

varya

Но дальше всё равно не понятно :-)

Теперь я хочу использовать в своём шаблоне блок link. Почему вот такой зависимости недостаточно ("ссылка" строится по дефолтным шаблонам): https://github.com/varya/varya.github.com/commit/bc4f0ef1fb2eae62b57f1c2d6f81fd713bb42dd6

10 years ago

tadatuta

потому что никто не зовет шаблон ссылки в клиентские шаблоны. должно быть:

[{
    mustDeps: 'i-bem'
 },
{
    tech: 'js',
    shouldDeps: { block: 'link', tech: 'bemhtml' }
}]

10 years ago

varya

А почему?? Ведь блок github независим, он должен строиться по-полному, и без разницы, откуда его позвали: из JavaSscript или BEMHTML. /cc @veged @dfilatov

10 years ago

tadatuta

Идея depsByTech предполагает, что ты можешь явно управлять тем, какие из шаблонов попадут в клиентский JS (иначе пришлось бы собирать все).

Когда блоки задекларированы без указания технологии, они собираются «один в один». В данном случае шаблон link попадает в index.bemhtml.js, но не попадает в index.min.js пока это не будет указано явно.

При этом нет необходимости из блока github указывать зависимость depsByTech от i-bem, сборщик поймет, что хотя link зависит от нее «обычным способом», она все равно нужна на клиенте.

10 years ago

varya

То есть если я использую библиотеку, в которой блок A строится с использованием блока B, и вдруг решила построить это всё на клиенте, мне нужно будет не только прописать зависимость depsByTech от блока A, но так же от блока B (и от всех других блоков, которые требуются для работы A)?

10 years ago

tadatuta

нет, если в блоке А прописана зависимость от В, то на клиент достаточно будет прописать только А.

10 years ago

varya

Но эта "зависимость A от B" должна быть byTech или обычная тоже сойдет?

10 years ago

apsavin

Должно работать с обычной.

При этом нет необходимости из блока github указывать зависимость depsByTech от i-bem, сборщик поймет, что хотя link зависит от нее «обычным способом», она все равно нужна на клиенте.

10 years ago

varya

Тогда возвращаюсь к своему вопросу "Почему вот такой зависимости не достаточно? https://github.com/varya/varya.github.com/commit/bc4f0ef1fb2eae62b57f1c2d6f81fd713bb42dd6"

10 years ago

apsavin

У тебя было указано в зависимостях через depsByTech, что шаблон github__repo.bemhtml должен попасть на клиент?

10 years ago

varya

Да, вот https://github.com/varya/varya.github.com/blob/410b359d0dc7f9abc8bbe321439a304651a9d2bd/desktop.blocks/github/github.deps.js#L9

10 years ago

qfox

See also: #111

10 years ago

apsavin

@zxqfox У Вари и так массив, разве нет?

10 years ago

tadatuta

@varya упячка вот здесь — из-за того, что тут используется mustDeps, многие блоки (и link в том числе) оказываются раньше, чем базовые шаблоны. соответственно, базовые шаблоны «перевешивают» шаблоны блоков.

10 years ago

varya

@tadatuta, Да вроде нет. Если переделать на shouldDeps — ничего не меняется, кроме того что highlight перестаёт работать как задумано :-)

10 years ago

qfox

@apsavin Просто ссылку оставил ;) вообще, я другой пост искал, и не факт, что он на форуме. Там где @veged, @floatdrop и @matmuchrapna обсуждали депсы.

10 years ago

tadatuta

@varya проверил, у меня после

diff --git a/desktop.blocks/i-bem/i-bem.deps.js b/desktop.blocks/i-bem/i-bem.deps.js
index 18e5f14..a97221a 100644
--- a/desktop.blocks/i-bem/i-bem.deps.js
+++ b/desktop.blocks/i-bem/i-bem.deps.js
@@ -1,5 +1,5 @@
 ({
-    mustDeps: [
+    shouldDeps: [
         {
             block: 'highlight',
             elem: 'hack'

шаблоны заработали как задумано.

10 years ago

varya

Спасибо. Я решила по-другому (но суть та же): Поскольку highlight__hack мне нужен рано, оставила mustDeps, но сделала это зависимостью page, а не i-bem. А у блока github прошу i-bem как mustDeps, чтобы он гарантированно был раньше.

Чтобы оставлять комментарии, необходимо авторизоваться.