fix: correct image size URLs in beforeChange (#15214)

### What
Fixes image size URLs in `beforeChange` pointing to original files
instead of their correct sized files.

### Why
The `beforeChange` hook was returning incorrect URLs for image size
variants. Plugins relying on the URLs were getting wrong file
references, causing issues for `plugin-cloud-storage`.

### Where
Updated the `beforeChange` in `getBaseFields` to use the image size URL.

### Testing
Added tests to `test/plugin-cloud-storage/int.spec.ts`

Author: jessrynkar

packages/payload/src/uploads/getBaseFields.ts

@@ -239,7 +239,11 @@ export const getBaseUploadFields = ({ collection, config }: Options): Field[] =>
                     generateFilePathOrURL({
                       collectionSlug: collection?.slug as string,
                       config,
-                      filename: data?.filename || originalDoc?.filename,
+                      filename:
+                        data?.sizes?.[size.name]?.filename ||
+                        originalDoc?.sizes?.[size.name]?.filename ||
+                        data?.filename ||
+                        originalDoc?.filename,
                       relative: true,
                       serverURL: req.payload.config.serverURL,
                       urlOrPath: value,

test/plugin-cloud-storage/int.spec.ts

@@ -152,6 +152,80 @@ describe('@payloadcms/plugin-cloud-storage', () => {
 
         expect($metadata.httpStatusCode).toBe(200)
       })
+
+      it('should store correct URLs for sized images', async () => {
+        const upload = await payload.create({
+          collection: mediaSlug,
+          data: {},
+          filePath: path.resolve(dirname, '../uploads/image.png'),
+        })
+
+        const apiResponse = await payload.findByID({
+          collection: mediaSlug,
+          id: upload.id,
+        })
+        expect(apiResponse.sizes).toBeTruthy()
+
+        const apiSizeKeys = Object.keys(apiResponse.sizes || {})
+        for (const sizeKey of apiSizeKeys) {
+          const size = apiResponse.sizes?.[sizeKey as keyof typeof apiResponse.sizes]
+          if (!size) {
+            continue
+          }
+
+          expect(size.url).toEqual(`/api/${mediaSlug}/file/${size.filename}`)
+        }
+
+        const rawDbData = await payload.db.findOne({
+          collection: mediaSlug,
+          where: { id: { equals: upload.id } },
+        })
+        expect(rawDbData).toBeTruthy()
+
+        const dbRecord = rawDbData as unknown as {
+          filename: string
+          sizes: Record<string, { filename: string; url: string }>
+          url: string
+        }
+        type SizeData = { filename: string; url: string }
+
+        const sizeKeys = Object.keys(dbRecord.sizes)
+        expect(sizeKeys.length).toBeGreaterThan(0)
+
+        for (const sizeKey of sizeKeys) {
+          const size: SizeData = dbRecord.sizes[sizeKey] as SizeData
+          expect(size.url).not.toEqual(`/api/${mediaSlug}/file/${dbRecord.filename}`)
+          expect(size.url).toEqual(`/api/${mediaSlug}/file/${size.filename}`)
+        }
+      })
+
+      it('should handle collections without imageSizes correctly', async () => {
+        const upload = await payload.create({
+          collection: mediaWithPrefixSlug,
+          data: {},
+          filePath: path.resolve(dirname, '../uploads/image.png'),
+        })
+
+        expect(upload.filename).toBeTruthy()
+        expect(upload.url).toEqual(`/api/${mediaWithPrefixSlug}/file/${upload.filename}`)
+        expect((upload as any).sizes).toBeFalsy()
+
+        const rawDbData = await payload.db.findOne({
+          collection: mediaWithPrefixSlug,
+          where: { id: { equals: upload.id } },
+        })
+
+        expect(rawDbData).toBeTruthy()
+
+        const dbRecord = rawDbData as unknown as {
+          filename: string
+          url: string
+        }
+
+        expect(dbRecord.filename).toEqual(upload.filename)
+        expect(dbRecord.url).toEqual(`/api/${mediaWithPrefixSlug}/file/${upload.filename}`)
+        expect((rawDbData as any)?.sizes).toBeFalsy()
+      })
     })
   })