ns.update и ns.view: отображение ошибок

[chestozo]

Без этих фиксов:

• не загружается страница, если для синхронных view пришли невалидные модели
• не рендерится состояние view ns-view-error-content

У нас - просто ничего не происходит (ns.Update тихо фейлится и всё).

[chestozo]

Поменялась только это строка, остальное отступ (можно по отдельным коммитам в PR это посмотреть).

[chestozo]

Поговорили голосом с @doochik, постараюсь за выходные допилить:

1. добавить метод ns.Update.handleError(models, modelErrors).
   Он должен возвращать true / false (по умолчанию возвращать будет false, чтобы не ломать совместимость с тем, как работает сейчас).
   true - всё хорошо, надо продолжить update и генерить html.
   false - оставить всё как есть - ну т.е. update зафейлился.

Хотели ещё в событии ns-page-before-load передавать promise от ns.Update, но мне вот сейчас кажется, что это всё можно сделать в ns.Update.handleError теперь. Предлагаю не делать )

Про обработку ошибок:
получается теперь приложение может отреагировать на полный треш (когда handlerError возвращет false) и на полутреш - рендерить view в состоянии ошибки (для этого даже придусмотрена сейчас специальная мода в yate - ns-view-error-content).

/cc @edoroshenko

[chestozo]

Да, в шаблон буду передавать ещё не только .models, но и .modelErrors, тут https://github.com/yandex-ui/noscript/blob/master/src/ns.view.js#L1078

[chestozo]

В общем, теперь можно переопределить метод обработки ошибок в ns.Update like this:

ns.Update.handleError = function(error, update) {
    if (update.isGlobal() && error.error === ns.U.STATUS.MODELS) {
        ns.page.go(ns.router.generateUrl('not-found', {}));
    }
    return false;
};

Надо ещё тесты, конечно, написать, а так готово!
Замечания / предложения?

[chestozo]

Up

[edoroshenko]

Странно, что эти изменения здесь видны. Они же уже в master

[chestozo]

Мне пришлось сюда их тоже замёржить, потому что мы не эту ветку смотрим у себя в проекте (нам нужен был этот патч).

[chestozo]

@doochik обещал ещё сегодня посмотреть )

[doochik]

У тебя в jsdoc написано string

[chestozo]

Это ваще не моё ) это я Игоря коммит подмёржил, потому что у нас проект на эту ветку смотрит сейчас )
Давай это отдельно потом поправим?

[chestozo]

up
/cc @edoroshenko @doochik

[chestozo]

Hooray )