GuzhavinAleksey/Vmeda.Online
1
0
Fork 0
forked from CIT/Vmeda.Online
Code Pull Requests Activity

Merge pull request #1820 from albertgasset/MOBILE-2915

Browse Source 
MOBILE-2915 site: Fix clearing of cached request promises
This commit is contained in:
Juan Leyva 2019-03-26 10:39:04 +01:00 committed by GitHub
commit c0697e4436
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/classes/site.ts
View File

@ -582,7 +582,7 @@ export class CoreSite {
});
}
let promise = this.getFromCache(method, data, preSets, false, originalData).catch(() => {
const promise = this.getFromCache(method, data, preSets, false, originalData).catch(() => {
// Do not pass those options to the core WS factory.
return this.wsProvider.call(method, data, wsPreSets).then((response) => {
if (preSets.saveToCache) {
@ -690,14 +690,12 @@ export class CoreSite {
this.ongoingRequests[cacheId] = promise;
// Clear ongoing request after setting the promise (just in case it's already resolved).
promise = promise.finally(() => {
return promise.finally(() => {
// Make sure we don't clear the promise of a newer request that ignores the cache.
if (this.ongoingRequests[cacheId] === promise) {
delete this.ongoingRequests[cacheId];
}
});
return promise.then((response) => {
}).then((response) => {
// We pass back a clone of the original object, this may prevent errors if in the callback the object is modified.
return this.utils.clone(response);
});