[CORD-1001] Solved race condition in autogeneration of models and tweaks for slow connections
Change-Id: Ifcbc4f9057f04e19703af96f8d8294f2b975be66
diff --git a/src/app/datasources/helpers/model-discoverer.service.ts b/src/app/datasources/helpers/model-discoverer.service.ts
index da6f265..434f74e 100644
--- a/src/app/datasources/helpers/model-discoverer.service.ts
+++ b/src/app/datasources/helpers/model-discoverer.service.ts
@@ -34,10 +34,12 @@
'ConfigHelpers',
'XosRuntimeStates',
'XosNavigationService',
- 'XosModelStore'
+ 'XosModelStore',
+ 'ngProgressFactory'
];
private xosModels: IXosModel[] = []; // list of augmented model definitions;
private xosServices: string[] = []; // list of loaded services
+ private progressBar;
constructor (
private $log: ng.ILogService,
@@ -46,8 +48,11 @@
private ConfigHelpers: IXosConfigHelpersService,
private XosRuntimeStates: IXosRuntimeStatesService,
private XosNavigationService: IXosNavigationService,
- private XosModelStore: IXosModelStoreService
+ private XosModelStore: IXosModelStoreService,
+ private ngProgressFactory: any // check for type defs
) {
+ this.progressBar = this.ngProgressFactory.createInstance();
+ this.progressBar.setColor('#f6a821');
}
public get(modelName: string): IXosModel|null {
@@ -56,7 +61,7 @@
public discover() {
const d = this.$q.defer();
-
+ this.progressBar.start();
this.XosModelDefs.get()
.then((modelsDef: IXosModeldef[]) => {
@@ -81,23 +86,28 @@
})
.then(model => {
this.$log.debug(`[XosModelDiscovererService] Model ${model.name} stored`);
- return this.$q.resolve();
+ return this.$q.resolve('true');
})
.catch(err => {
this.$log.error(`[XosModelDiscovererService] Model ${model.name} NOT stored`);
- // NOTE why this does not resolve?????
- // return this.$q.resolve();
- return this.$q.reject();
+ return this.$q.resolve('false');
});
pArray.push(p);
});
this.$q.all(pArray)
- .then(() => {
+ .then((res) => {
+ // the Model Loader promise won't ever be reject, in case it will be resolve with value false,
+ // that's because we want to wait anyway for all the models to be loaded
+ if (res.indexOf('false') > -1) {
+ d.resolve(false);
+ }
d.resolve(true);
- this.$log.info('[XosModelDiscovererService] All models loaded!');
})
.catch(() => {
d.resolve(false);
+ })
+ .finally(() => {
+ this.progressBar.complete();
});
});
return d.promise;
@@ -210,19 +220,19 @@
private cacheModelEntries(model: IXosModel): ng.IPromise<IXosModel> {
const d = this.$q.defer();
- let called = false;
+ let populated = false;
const apiUrl = this.getApiUrlFromModel(model);
this.XosModelStore.query(model.name, apiUrl)
.subscribe(
() => {
// skipping the first response as the observable gets created as an empty array
- if (called) {
+ if (populated) {
return d.resolve(model);
}
- called = true;
+ populated = true;
},
err => {
- d.reject(err);
+ return d.reject(err);
}
);
diff --git a/src/app/datasources/stores/model.store.ts b/src/app/datasources/stores/model.store.ts
index 4958015..61eb114 100644
--- a/src/app/datasources/stores/model.store.ts
+++ b/src/app/datasources/stores/model.store.ts
@@ -24,10 +24,15 @@
public query(modelName: string, apiUrl: string): Observable<any> {
// if there isn't already an observable for that item
+ // create a new one and .next() is called by this.loadInitialData once data are received
if (!this._collections[modelName]) {
this._collections[modelName] = new BehaviorSubject([]); // NOTE maybe this can be created when we get response from the resource
this.loadInitialData(modelName, apiUrl);
}
+ // else manually trigger the next with the last know value to trigger the subscribe method of who's requestiong this data
+ else {
+ this._collections[modelName].next(this._collections[modelName].value);
+ }
this.webSocket.list()
.filter((e: IWSEvent) => e.model === modelName)
@@ -76,7 +81,7 @@
}
private loadInitialData(model: string, apiUrl?: string) {
- // TODO provide alway the apiUrl togheter with the query() params
+ // TODO provide always the apiUrl togheter with the query() params
if (!angular.isDefined(apiUrl)) {
// NOTE check what is the correct pattern to pluralize this
apiUrl = this.storeHelpers.urlFromCoreModel(model);
@@ -87,10 +92,8 @@
this._collections[model].next(res);
})
.catch(
- // TODO understand how to send an error to an observable
err => {
this._collections[model].error(err);
- // this.$log.log(`Error retrieving ${model}`, err);
}
);
}