Commit 70e59559 authored by Filipa Lacerda's avatar Filipa Lacerda

Clears timeout

parent 14b077b7
...@@ -36,6 +36,7 @@ export default class Poll { ...@@ -36,6 +36,7 @@ export default class Poll {
this.options.data = options.data || {}; this.options.data = options.data || {};
this.intervalHeader = 'POLL-INTERVAL'; this.intervalHeader = 'POLL-INTERVAL';
this.timeoutID = null;
this.canPoll = true; this.canPoll = true;
} }
...@@ -44,11 +45,8 @@ export default class Poll { ...@@ -44,11 +45,8 @@ export default class Poll {
const pollInterval = headers[this.intervalHeader]; const pollInterval = headers[this.intervalHeader];
if (pollInterval > 0 && response.status === httpStatusCodes.OK && this.canPoll) { if (pollInterval > 0 && response.status === httpStatusCodes.OK && this.canPoll) {
setTimeout(() => { this.timeoutID = setTimeout(() => {
// Stop can be called in the meanwhile, so let's check again. this.makeRequest();
if (this.canPoll) {
this.makeRequest();
}
}, pollInterval); }, pollInterval);
} }
...@@ -63,7 +61,13 @@ export default class Poll { ...@@ -63,7 +61,13 @@ export default class Poll {
.catch(error => errorCallback(error)); .catch(error => errorCallback(error));
} }
/**
* Stops the polling recursive chain
* and guarantees if the timeout is already running it won't make another request by
* cancelling the previously established timeout.
*/
stop() { stop() {
this.canPoll = false; this.canPoll = false;
clearTimeout(this.timeoutID);
} }
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment