Передача контекста блока внутрь функции для использования this.* about bem-bl

[indutny]

Should be fixed by #560, could you please verify?

[belyanskii]

@indutny проверил с веткой из PR - в собранном bemhtml.js ничего не поменялось кажется и проблема остается. Если что - код для проверки тут, в bower.json просто можешь свою ветку ставить и смотреть :)

[indutny]

Это потому что ты используешь bem-bl-xjst, в нем есть своя копия bem-bl. Сейчас сделаю и туда PR.

[indutny]

bem-archive/bem-bl-xjst#5

[belyanskii]

@indutny - попробовал с bem-bl-xjst из ветки что в PR, тоже ничего не полетело, возможно еще что-нибудь надо поменять? (enb-xjst версию?)

[indutny]

Все ясно, у тебя теперь там другой пример :) Вопрос же был про чтение из this, а теперь ты туда пишешь и даже вызываешь apply. cc @veged

[belyanskii]

@indutny да, эту проблему тоже обсуждали и казалось что оно связанно

[veged]

там вроде только apply и вызывается? что мешает также сделать как и с чтением из this? если есть apply то прокидывать честный объект

[indutny]

@veged я придумал как сделать, скоро починим :)

[belyanskii]

@indutny ping :)

[belyanskii]

@indutny pong

[tadatuta]

@indutny Am I right that the fix should also be ported to bem-xjst?

[indutny]

I'd wait for a whole fix. Right now I have fixed only part of it in these commits and in veged/xjst@e4cffd6 . Going to finish this up tomorrow.

[indutny]

@belyanskii please give a try to latest: bem-archive/bem-bl-xjst#5

[belyanskii]

@indutny it's working!! brilliant!! @tadatuta, now i need new versions of library's 👍

[tadatuta]

@indutny Fedor, will you release bem-bl-xjst yourself or should I do it while you're on vacation? :)
To do so I just need to know how destructive these changes are: should it be major version or patch will be enough? And should it be ported into support/1.x branch as well?

And what about bem-xjst? Do we need any changes there or weak deps on xjst is just enough?

[indutny]

@tadatuta I'll do it in a patch version, and yeah it should be ported.

[indutny]

@belyanskii released in [email protected]

[indutny]

@tadatuta ah, it seems like support/1.x is not affected, because it isn't using this optimisation.