@uppy/core: pass file to events consistently

[Murderlon]

Closes #3938

[github-actions]

Diff output files

diff --git a/packages/@uppy/companion-client/lib/RequestClient.js b/packages/@uppy/companion-client/lib/RequestClient.js
index a7ef904..6dbea5e 100644
--- a/packages/@uppy/companion-client/lib/RequestClient.js
+++ b/packages/@uppy/companion-client/lib/RequestClient.js
@@ -466,8 +466,8 @@ async function _awaitRemoteFileUpload2(_ref4) {
         this.uppy.log(`upload ${file.id} was canceled`, "info");
         resolve();
       };
-      const onFilePausedChange = (targetFileId, newPausedState) => {
-        if (targetFileId !== file.id) return;
+      const onFilePausedChange = (targetFile, newPausedState) => {
+        if ((targetFile == null ? void 0 : targetFile.id) !== file.id) return;
         pause(newPausedState);
       };
       const onPauseAll = () => pause(true);
diff --git a/packages/@uppy/core/lib/EventManager.js b/packages/@uppy/core/lib/EventManager.js
index 05ee7f4..54d8574 100644
--- a/packages/@uppy/core/lib/EventManager.js
+++ b/packages/@uppy/core/lib/EventManager.js
@@ -32,8 +32,8 @@ export default class EventManager {
     }
   }
   onFilePause(fileID, cb) {
-    this.on("upload-pause", (targetFileID, isPaused) => {
-      if (fileID === targetFileID) {
+    this.on("upload-pause", (file, isPaused) => {
+      if (fileID === (file == null ? void 0 : file.id)) {
         cb(isPaused);
       }
     });
@@ -44,15 +44,15 @@ export default class EventManager {
     });
   }
   onPause(fileID, cb) {
-    this.on("upload-pause", (targetFileID, isPaused) => {
-      if (fileID === targetFileID) {
+    this.on("upload-pause", (file, isPaused) => {
+      if (fileID === (file == null ? void 0 : file.id)) {
         cb(isPaused);
       }
     });
   }
   onRetry(fileID, cb) {
-    this.on("upload-retry", targetFileID => {
-      if (fileID === targetFileID) {
+    this.on("upload-retry", file => {
+      if (fileID === (file == null ? void 0 : file.id)) {
         cb();
       }
     });
diff --git a/packages/@uppy/core/lib/Uppy.js b/packages/@uppy/core/lib/Uppy.js
index 3b2edee..8bbd650 100644
--- a/packages/@uppy/core/lib/Uppy.js
+++ b/packages/@uppy/core/lib/Uppy.js
@@ -632,12 +632,13 @@ export class Uppy {
     if (!this.getState().capabilities.resumableUploads || this.getFile(fileID).progress.uploadComplete) {
       return undefined;
     }
-    const wasPaused = this.getFile(fileID).isPaused || false;
+    const file = this.getFile(fileID);
+    const wasPaused = file.isPaused || false;
     const isPaused = !wasPaused;
     this.setFileState(fileID, {
       isPaused,
     });
-    this.emit("upload-pause", fileID, isPaused);
+    this.emit("upload-pause", file, isPaused);
     return isPaused;
   }
   pauseAll() {
@@ -698,7 +699,7 @@ export class Uppy {
       files: updatedFiles,
       error: null,
     });
-    this.emit("retry-all", filesToRetry);
+    this.emit("retry-all", Object.values(updatedFiles));
     if (filesToRetry.length === 0) {
       return Promise.resolve({
         successful: [],
@@ -733,7 +734,7 @@ export class Uppy {
       error: null,
       isPaused: false,
     });
-    this.emit("upload-retry", fileID);
+    this.emit("upload-retry", this.getFile(fileID));
     const uploadID = _classPrivateFieldLooseBase(this, _createUpload)[_createUpload]([fileID], {
       forceAllowNewUpload: true,
     });
@@ -1444,10 +1445,7 @@ function _createUpload2(fileIDs, opts) {
     throw new Error("Cannot create a new upload: already uploading.");
   }
   const uploadID = nanoid();
-  this.emit("upload", {
-    id: uploadID,
-    fileIDs,
-  });
+  this.emit("upload", uploadID, this.getFilesByIds(fileIDs));
   this.setState({
     allowNewUpload: this.opts.allowMultipleUploadBatches !== false && this.opts.allowMultipleUploads !== false,
     currentUploads: {

[mifi]

Leaving upload and retry-all events as IDs because it might be a lot of files and it's less likely you want to do something with all files there.

did we agree to also convert these for consistency?

[Murderlon]

did we agree to also convert these for consistency?

Done