From 9dcef7bb41d73f757615a60d1bd70cbab108441b Mon Sep 17 00:00:00 2001 From: Murderlon Date: Tue, 9 Aug 2022 15:46:39 +0200 Subject: [PATCH] Add check for file object vs file id --- packages/@uppy/aws-s3/src/MiniXHRUpload.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/@uppy/aws-s3/src/MiniXHRUpload.js b/packages/@uppy/aws-s3/src/MiniXHRUpload.js index b2a5cded79..3483fdbca2 100644 --- a/packages/@uppy/aws-s3/src/MiniXHRUpload.js +++ b/packages/@uppy/aws-s3/src/MiniXHRUpload.js @@ -100,8 +100,12 @@ export default class MiniXHRUpload { } #addEventHandlerForFile (eventName, fileID, eventHandler) { - this.uploaderEvents[fileID].on(eventName, (targetFileID) => { - if (fileID === targetFileID) eventHandler() + this.uploaderEvents[fileID].on(eventName, (fileOrID) => { + // TODO (major): refactor Uppy events to consistently send file objects (or consistently IDs) + // We created a generic `addEventListenerForFile` but not all events + // use file IDs, some use files, so we need to do this weird check. + const id = fileOrID?.id ?? fileOrID + if (fileID === id) eventHandler() }) }