Skip to content

Commit ce71cba

Browse files
authored
Merge pull request #842 from RZulfikri/fix/download-event
add condition inside download event, before pass it to event callback.
2 parents aa079a5 + 762d29d commit ce71cba

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

FS.common.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -500,15 +500,21 @@ var RNFS = {
500500
var subscriptions = [];
501501

502502
if (options.begin) {
503-
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadBegin', options.begin));
503+
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadBegin', (res) => {
504+
if (res.jobId === jobId) options.begin(res);
505+
}));
504506
}
505507

506508
if (options.progress) {
507-
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadProgress', options.progress));
509+
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadProgress', (res) => {
510+
if (res.jobId === jobId) options.progress(res);
511+
}));
508512
}
509513

510514
if (options.resumable) {
511-
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadResumable', options.resumable));
515+
subscriptions.push(RNFS_NativeEventEmitter.addListener('DownloadResumable', (res) => {
516+
if (res.jobId === joibId) options.resumable(res);
517+
}));
512518
}
513519

514520
var bridgeOptions = {

0 commit comments

Comments
 (0)