search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

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

Open chestozo opened 7 years ago

chestozo commented 7 years ago

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

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

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

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

vitkarpov commented 7 years ago

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

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

chestozo commented 7 years ago

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

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

vitkarpov commented 7 years ago

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

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

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

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

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

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

Виктор.

chestozo commented 7 years ago

Ну тут 2 темы:

vitkarpov commented 7 years ago

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

+1 — про это issue

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

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

chestozo commented 7 years ago

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

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

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

vitkarpov commented 7 years ago

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

Вообще, похоже на https://github.com/yandex-ui/noscript/issues/619 Я аналогично в patchLayout хотел бы дотюнить

vitkarpov commented 7 years ago

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

chestozo commented 7 years ago

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