[ns.Update] Перезапрос моделей в случае их невалидности #464

Author: doochik

src/ns.update.js

@@ -42,6 +42,13 @@
 
         this.promise = new Vow.Promise();
 
+        /**
+         * Количество перезапросов из-за невалидных моделей.
+         * @type {number}
+         * @private
+         */
+        this._restartCount = 1;
+
         options = options || {};
 
         /**
@@ -85,6 +92,13 @@
      */
     ns.Update.prototype._EVENTS_ORDER = ['ns-view-hide', 'ns-view-htmldestroy', 'ns-view-htmlinit', 'ns-view-async', 'ns-view-show', 'ns-view-touch'];
 
+    /**
+     * Лимит на перезапрос моделей.
+     * @type {number}
+     * @constant
+     */
+    ns.Update.prototype.RESTART_LIMIT = 2;
+
     /**
      * Регистрирует указанное событие, добавляя к нему признаки ns.update
      * @private
@@ -153,7 +167,7 @@
         };
 
         if (ns.Update.handleError(error, this)) {
-            return [].concat(err.invalid, err.valid);
+            return Vow.resolve([].concat(err.invalid, err.valid));
 
         } else {
             return Vow.reject(error);
@@ -202,6 +216,12 @@
         this.log('collected incomplete views', views);
 
         var models = views2models(views.sync);
+        /**
+         * Массив моделей, которые должны быть валидны перед отрисовкой.
+         * @private
+         */
+        this._models = models;
+
         this.log('collected needed models', models);
         this.stopTimer('collectModels');
 
@@ -340,11 +360,67 @@
      * потому что у видов будет уже другое состояние, если что-то поменяется между generateHTML и insertNodes
      * @private
      */
-    ns.Update.prototype._updateDOM = function() {
+    ns.Update.prototype._startUpdateDOM = function() {
         if (this._expired()) {
             return this._rejectWithStatus(this.STATUS.EXPIRED);
         }
 
+        // Проверяем валидность моделей перед стартом.
+        // Из-за асинхронности модели могут оказаться невалидными.
+        var models = {
+            valid: [],
+            invalid: []
+        };
+        var modelHasErrors = false;
+        for (var i = 0, j = this._models.length; i < j; i++) {
+            var model = this._models[i];
+            if (model.isValid()) {
+                models.valid.push(model);
+
+            } else {
+                models.invalid.push(model);
+                if (model.status === model.STATUS.ERROR) {
+                    modelHasErrors = true;
+                    break;
+                }
+            }
+        }
+
+        // если все хорошо, то переходим к обновлению DOM
+        if (models.invalid.length === 0) {
+            return this._updateDOM();
+        }
+
+        // если в моделях есть ошибки или мы превысили лимит перезапусков, то фейлимся
+        if (modelHasErrors || this._restartCount >= this.RESTART_LIMIT) {
+            this.log('update has bad models', this._restartCount, models);
+            // имитируем ошибку "Невалидные модели"
+            var promise = this._onRequestModelsError(models);
+            // делаем это синхронно
+            if (promise.isFulfilled()) {
+                ns.log.info('ns.Update.error-content', {
+                    _restartCount: this._restartCount
+                }, models.invalid);
+                // если ns.Update.handleError решит, что все ок, то запускаем обновление DOM
+                return this._updateDOM();
+
+            } else {
+                return promise;
+            }
+
+        } else {
+            // если модели невалидные, но в них нет ошибок, то пробуем перезапуститься
+            this._restartCount++;
+            this.log('restart request models', this._restartCount);
+            ns.log.info('ns.Update.restart', {
+                _restartCount: this._restartCount
+            }, models.invalid);
+            return this._requestModels(this._models)
+                .then(this._startUpdateDOM, null, this);
+        }
+    };
+
+    ns.Update.prototype._updateDOM = function() {
         var html = this._renderUpdateTree();
         var node = ns.html2node(html || '');
         return this._insertNodes(node);
@@ -421,7 +497,7 @@
         // начинаем цепочку с промиса, чтобы ловить ошибки в том числе и из _requestAllModels
         Vow.invoke(this._requestAllModels.bind(this))
             .then(saveAsyncPromises, null, this)
-            .then(this._updateDOM, null, this)
+            .then(this._startUpdateDOM, null, this)
             .then(fulfillWithAsyncPromises, this._reject, this);
 
         return this.promise;

test/spec/ns.update.edge-cases.js

@@ -206,4 +206,5 @@ describe('ns.Update. Синтетические случаи', function() {
                 expect(ns.request.models).to.have.callCount(0);
             });
     });
+
 });

test/tests.yate

@@ -20,6 +20,7 @@ match .* ns-view-async-content {
 
 match .* ns-view-error-content {
     "test ns-view-error-content"
+    apply . ns-view-content
 }
 
 // Это шаблон для проверки того, что yate сбилжен для тестов.

yate/noscript.yate

@@ -114,4 +114,8 @@ match .* ns-view-async-content {}
 
 // Часть обязательных моделей не удалось получить или для них вернулась ошибка
 // @public
-match .* ns-view-error-content {}
+match .* ns-view-error-content {
+    // Надо все равно отрисовать дерево,
+    // чтобы не нарваться на ошибку "Can't find node for <view>"
+    apply . ns-view-content
+}