Нужно абортить ns.Update если в момент начала отрисовки у модели state=invalid

[chestozo]

В ns.Update есть такой код

Vow.invoke(this._requestAllModels.bind(this))
    .then(function(result) {
        // >>> Каким-то образом тут у какой-то из модели получаем state='invalid'
        // >>> И мы отрендерим error моду для вида.
        this._updateDOM();
        this._fulfill(result);
    }, this._reject, this)
    .then(null, this._reject, this);

У нас есть стабильный кейс (напишу тест), когда глобальный update запускается и в момент отрисовки одна из моделей в состоянии invalid.
Update считает, что модель в состоянии ошибки и рендерит error моду для вида.
Но у нас уже запущен (или вот вот запустится) другой update и всё перерисует.

Кажется, тут надо abort-ить update, а не рисовать error моду иначе всё моргает в интерфейсе.

[vitkarpov]

Кажется, тут надо abort-ить update, а не рисовать error моду иначе всё моргает в
интерфейсе.

Я ж правильно понял, что, вообще говоря, error-мода вьюшки должна рисоваться. Т.е. если следующий апдейт не запустится или в следующем апдейте состояние модели не изменится?

[chestozo]

По идее, error мода нужна для состояния, когда модель в ошибке (state=error).
А тут у нас модель в состоянии "меня проинвалидировали" (state=invalid) - кто-то вызвал model.invalidate().

Если считать, что этот кто-то, кто вызывал invalidate затем запустит ns.page.go() - то всё будет хорошо.

Итого предложение:
Было

• рендерился вид в состоянии ошибки
• очередной update приводил к нормальной перерисовке (очень часто это выглядит как моргание)

Стало

• абортим update если одна из моделей с state=invalid
• очередной update приводит к нормальной перерисовке (и по идее ничего не моргнёт)

[vitkarpov]

Кул!

[chestozo]

Демо видео https://cl.ly/1q38461f1x2j

[vitkarpov]

Замутил фикс? :)

[chestozo]

У меня есть одна идея фикса, но она мне не до конца нравится..
Идея в том, чтобы кидать exception в ns.View.prototype._getModelsForTree если model.status === 'invalid' - я не до конца понимаю, насколько это безопасно.

[chestozo]

Пилю тут #650

[chestozo]

Один из вариантов обхода - не вызывать совсем model.invalidate(), вместо этого выполнять ns.forcedRequest.

[vitkarpov]

А не инвалидировать модель где?

[chestozo]

Ну где-то в коде ) у нас просто случай ручной инвалидации..

[vitkarpov]

под «где» я подразумевал на стороне приложения, да. уточнил, что я правильно тебя понял. но ведь это не решает проблему, в целом, верно? в следующий раз опять кто-то инвалидирует и будет париться что ж там случилось?

[chestozo]

да ( тут есть идея, что может ок рисовать вид с невалидной моделью?
Потому что когда-то недавно эти данные были валидными.
Ну т.е. у модели есть данные, просто status=invalid.
Что скажешь? Ну т.е. даже update не отменять

[vitkarpov]

тут есть идея, что может ок рисовать вид с невалидной моделью?

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

[vitkarpov]

Что скажешь? Ну т.е. даже update не отменять

Я соглашусь с тем, что вот прямо сейчас это место переделать может быть довольно сложно. Это ок.

Я сейчас больше про концепцию и признание того, что баг в каком-то виде в этом месте есть.

[chestozo]

Точно есть )

В общем варианта по идее 2:

• абортить update
• рендерить так, как будто всё хорошо )

По идее, оба должны привести к тому, что моргать не будет.