Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

apply & applyNext description fix Close #676 #677

Merged
merged 2 commits into from
Sep 19, 2014
Merged

apply & applyNext description fix Close #676 #677

merged 2 commits into from
Sep 19, 2014

Conversation

Murdalay
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ba4529d on issues/#676@v2 into e45dec2 on v2.


При вызове `applyNext` выполняются следующие шаги:

1. Создание в контексте флага, позволяющего избежать бесконечной рекурсии при вызове шаблонов. В качестве флага
используется случайное число.
2. Добавление в предикат шаблона проверки на наличие флага.
3. Выполнение блока `expressions` (модификация текущего контекста).
4. Вызов процедуры выбора и выполнения шаблона `apply()`.
3. Установка полей контекста, определенных в объекте `{assignObj}`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лишний спейс после «в объекте»

@tadatuta
Copy link
Member

👍


```js
block('b1')(function(){ return 'Ohoy!' })
block('b1')(applyNext()) // Ohoy!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут не хватает какой-нить моды и return с функцией во втором шаблоне

block('b1').tag()('span')
block('b1').tag()(function() { return applyNext(); }) // span

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 04fb6a0 on issues/#676@v2 into e45dec2 on v2.

tadatuta added a commit that referenced this pull request Sep 19, 2014
apply & applyNext description fix Close #676
@tadatuta tadatuta merged commit 351f3b0 into v2 Sep 19, 2014
@tadatuta tadatuta deleted the issues/#676@v2 branch September 19, 2014 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants