EVOgeek
/
Vmeda.Online
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1411 from albertgasset/MOBILE-2482 

MOBILE-2482 core: Fix concurrent delegate handler updates
main
Juan Leyva 2018-07-12 11:22:34 +02:00 committed by GitHub
parent 814843a1e6 463d78a3b3
commit 5f614a5677
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/classes/delegate.ts
View File

@ -252,9 +252,8 @@ export class CoreDelegate {
this.updatePromises[siteId][handler.name] = promise.catch(() => { this.updatePromises[siteId][handler.name] = promise.catch(() => {
return false; return false;
}).then((enabled: boolean) => { }).then((enabled: boolean) => {
// Verify that this call is the last one that was started.
// Check that site hasn't changed since the check started. // Check that site hasn't changed since the check started.
if (this.isLastUpdateCall(time) && this.sitesProvider.getCurrentSiteId() === siteId) { if (this.sitesProvider.getCurrentSiteId() === siteId) {
if (enabled) { if (enabled) {
this.enabledHandlers[handler[this.handlerNameProperty]] = handler; this.enabledHandlers[handler[this.handlerNameProperty]] = handler;
} else { } else {