grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Align api.subscribe with contract

Browse Source
This commit is contained in:
Jaco Greeff 2016-12-09 00:15:39 +01:00
parent 4ce3142c63
commit 1ecda93de9
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
js/src/api/subscriptions/manager.js
View File

@ -82,7 +82,11 @@ export default class Manager {
if (!engine.isStarted) {
engine.start();
} else {
this._sendData(subscriptionId, error, data);
const result = this._sendData(subscriptionId, error, data);
if (autoRemove && !result) {
this.unsubscribe(subscriptionId);
}
}
resolve(subscriptionId);
@ -102,19 +106,18 @@ export default class Manager {
}
_sendData (subscriptionId, error, data) {
const { callback } = this.subscriptions[subscriptionId];
let result = false;
try {
return callback(error, data);
result = this.subscriptions[subscriptionId](error, data);
} catch (error) {
console.error(`Unable to update callback for subscriptionId ${subscriptionId}`, error);
}
return true;
return result;
}
_updateSubscriptions = (subscriptionName, error, data) => {
const cleanup = [];
const subscriptions = this.subscriptions
.filter(subscription => subscription.name === subscriptionName);
@ -125,13 +128,9 @@ export default class Manager {
const result = this._sendData(subscription.id, error, data);
if (subscription.autoRemove && !result) {
cleanup.push(subscription.id);
this.unsubscribe(subscription.id);
}
});
cleanup.forEach((subscriptionId) => {
delete this.subscriptions[subscriptionId];
});
}
}