Перезапрос моделей в ns.Update #464

Author: doochik

src/ns.view.js

@@ -665,29 +665,46 @@
          */
         this.asyncState = false;
 
+        /*
+         Логика такая: все виды должны себя ВСЕГДА добавлять.
+         Иначе может получиться странная ситуация:
+         Есть вид v1, он зависит от модель m1. Он валиден.
+         Есть вид v2, он зависит от модели m2. Он невалиден.
+
+         v1 не сообщает свои модели, а v2 сообщает.
+         В ns.request уходят знания только про m2.
+
+         Допустим в процессе запроса m2 кто-то инвалидировал модель m1.
+         Про ее необходимость никто не знает,
+         соответственно дело нормальным способом дойдет до отрисовки,
+         где m1 будет перерисован как error-content
+         */
+
         if (this.async) {
             var hasValidModels = this.isModelsValid();
-            var hasValidStatus = this.isOk();
 
             // shouldBeSync - специальный флаг, чтобы вид отрисовался
             // раньше это работало так,
             // update параллельно запрашивал модели для синхронных видов и асинхронных
             // поэтому когда запускался update для асинхронных модели уже были готовы и hasValidModels === true
-            if (this.shouldBeSync || (hasValidModels && !hasValidStatus)) {
+            if (this.shouldBeSync || hasValidModels) {
                 // если асинхронный блок имеет валидные модели, но невалидный статус - рисуем его синхронно
                 updated.sync.push(this);
 
-            } else if (!hasValidModels) {
+            } else {
                 this.asyncState = true;
                 // если асинхронный блок имеет невалидные модели, то его не надо рисовать сразу
                 updated.async.push(this);
                 // прекращаем обработку
                 return updated;
             }
-        } else if (!this.isValidSelf()) {
-            // реинвалидируем дочерние виды,
-            // потому что их тоже придется перерисовать вместе с родителем
-            this.invalidate();
+        } else {
+
+            if (!this.isValidSelf()) {
+                // реинвалидируем дочерние виды,
+                // потому что их тоже придется перерисовать вместе с родителем
+                this.invalidate();
+            }
 
             // если обычный блок не валиден
             updated.sync.push(this);