Improving IngestionAdminT5 model fetch.

- better and user-friendlier error handling
- no-fetch before the view pops up (no errors for disabled ingestion)

Author: pacesm

app/scripts/controller/IngestionAdminT5Controller.js

@@ -50,16 +50,6 @@
 
             initialize: function(options) {
 
-                // model initialization:
-                this.model.fetch({
-                    error:function(m,r,o){
-                        console.error("Failed to fetch the ingestion scenarios!");
-                        console.error(m);
-                        console.error(r);
-                        console.error(o);
-                    }
-                });
-
                 // register events the component is listenning to:
                 this.listenTo(Communicator.mediator, "dialog:open:ingestionAdminT5", this.onIngestionAdminOpen);
                 this.listenTo(Communicator.mediator, "dialog:close:ingestionAdminT5", this.onIngestionAdminClose);

app/scripts/models/IngestionAdminT5Model.js

@@ -16,10 +16,11 @@
                 config_src.ingestionEngineT5.listScenario;
             var IngestionAdminT5Model = Backbone.Model.extend(
                 {
+                    failure: false,
                     url: url_src,
                     ToI:{},          // Time of Interest
                     AoI:[],          // Area of Interest
-                    scenarios: []    // Defined Scenarios
+                    scenarios: null  // Defined Scenarios
                 }
             );
 

app/scripts/views/IngestionAdminT5View.js

@@ -61,6 +61,22 @@
                 is_showing     : false,
                 change_pending : false,
 
+                fetch_model: function(){
+                    var that = this;
+                    this.model.fetch({
+                        suppressErrors: that.model.get('failure'),
+                        error: function() {
+                                that.model.set({
+                                    'failure': true,
+                                    'scenarios': null
+                                });
+                            },
+                        success: function() {
+                                that.model.set({'failure': false});
+                            }
+                    });
+                },
+
                 get_scenario_st : function(that, sc_id, ncn_id) {
                     var scenarios = that.model.get('scenarios');
                     for (var i=0; i<scenarios.length; i++)
@@ -151,7 +167,11 @@
                             alertMessage = "No ingestion scenario defined. Create a new one by clicking on the Create Scenario button below.";
                         }
                     } else {
-                        alertMessage = "Failed to fetch scenarios from the Ingestion Engine!";
+                        if ( that.model.get('failure') ) {
+                            alertMessage = "Failed to fetch scenarios from the Ingestion Engine!";
+                        } else {
+                            alertMessage = "Fetching the scenarios ...";
+                        }
                     }
 
                     that.$('#ingestion-alert').html(alertMessage);
@@ -219,10 +239,9 @@
                                         console.log('IngAdmT5: '+ncn_id+' ajax ' +op+ ' ' + errorThrown);
                                 }, this));
 
-                        var idel;
-                        idel = setInterval(_.bind(function() {
+                        var idel = setInterval(_.bind(function() {
                                     if (this.change_pending) {
-                                        this.model.fetch();
+                                        this.fetch_model();
                                     } else {
                                         clearInterval(idel);
                                     }
@@ -252,7 +271,7 @@
 
                     this.$('#ingestion-alert').html("Loading scenarios from the Ingestion Engine ...");
 
-                    this.updater.i = setInterval(_.bind(function() { this.model.fetch(); }, this ), 2250);
+                    this.updater.i = setInterval(_.bind(function() { this.fetch_model(); }, this ), 2250);
 
                     this.$el.draggable(
                     { 
@@ -378,10 +397,10 @@
 
                 onTest: function() 
                 {
-                    this.model.fetch();
+                    this.fetch_model();
 
                     this.updater.i = setInterval(_.bind(function() {
-                                this.model.fetch();
+                                this.fetch_model();
                             },
                             this ),
                         1250);