ns.modelCollection.js метод info.split.model_id вызывается в контексте window

[chestozo]

У нас в noscript есть возможность в секции split указать model_id в виде функции.
Сигнатура такая: model_id(itemData, itemParams).

Этот метод сейчас вызывается в контексте window.

Предложение:
давайте вызывать его в контексте модели-коллекции.

Зачем?
К примеру, чтобы подлезть в данные элемента коллекции или допилить его параметры.
Может запилим?

[vitkarpov]

К примеру, чтобы подлезть в данные элемента коллекции или допилить его параметры.

Она ж вроде и так принимает данные и параметры?

[chestozo]

давайте вызывать его в контексте модели-коллекции.

Вот это суть, чтобы this внутри метода было коллекцией.

[vitkarpov]

Не, это да. Ты просто пишешь про "подлезть в данные модели или параметры",
но они приодет в аргументах, поэтому я просто не понял зачем там нужен
контекст.

В целом, нормальная тема.

четверг, 18 августа 2016 г. пользователь chestozo написал:

давайте вызывать его в контексте модели-коллекции.

Вот это суть, чтобы this внутри метода было коллекцией.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#623 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC3Nvza0OdLDTBDq-c76XqW91t41gT1oks5qg4cbgaJpZM4JmkDi
.

Виктор.

[chestozo]

Ну тут 2 темы:

• хочется в this коллекцию, чтобы, к примеру, вызвать у неё метод и ...
• itemParams и itemsData в данном методе можно потюнить. Это хак и нефункциональненько, но мне уже хотелось так сделать однажды ;)

[vitkarpov]

хочется в this коллекцию, чтобы, к примеру, вызвать у неё метод и ...

+1 — про это issue

itemParams и itemsData в данном методе можно потюнить.

ты к тому, что их как раз можно будет потюнить через АПИ модельки? да, это норм

[chestozo]

itemParams и itemsData в данном методе можно потюнить.

ты к тому, что их как раз можно будет потюнить через АПИ модельки? да, это норм

ага ) норм, но не очень ожидаемо может быть )

[vitkarpov]

itemParams и itemsData в данном методе можно потюнить.

Вообще, похоже на #619
Я аналогично в patchLayout хотел бы дотюнить

[vitkarpov]

@chestozo запилишь в 0.8.8? Вроде не долго делать, да?
P.S. Если нет возможности — давай я запилю, ты поревьюишь (если так удобнее)

[chestozo]

Постараюсь запилить да )