fix(ui): bulk uploads silently skips files without filenames (#14621)

### What?

Fixes an issue where bulk uploading files with missing filenames would
silently skip without providing error feedback to the user.

### Why?

Files without filenames would attempt to upload, fail on the server, but
show no error to the user. Only the network tab would show the failure.

### How?

Check for missing filenames before upload and skip those files with an
error count. Trigger validation after upload to show "A file name is
required" error message. Failed files remain in the drawer for users to
fix.

Fixes #14512

Author: PatrikKozak

packages/ui/src/elements/BulkUpload/FormsManager/index.tsx

@@ -333,6 +333,21 @@ export function FormsManagerProvider({ children }: FormsManagerProps) {
       for (let i = 0; i < currentForms.length; i++) {
         try {
           const form = currentForms[i]
+          const fileValue = form.formState?.file?.value
+
+          // Skip upload if file is missing a filename
+          if (
+            fileValue &&
+            typeof fileValue === 'object' &&
+            'name' in fileValue &&
+            (!fileValue.name || fileValue.name === '')
+          ) {
+            currentForms[i] = {
+              ...currentForms[i],
+              errorCount: 1,
+            }
+            continue
+          }
 
           setLoadingText(t('general:uploadingBulk', { current: i + 1, total: currentForms.length }))
 

test/uploads/e2e.spec.ts

@@ -1269,6 +1269,90 @@ describe('Uploads', () => {
       // should show add files dropzone view
       await expect(fieldBulkUploadDrawer.locator('.bulk-upload--add-files')).toBeVisible()
     })
+
+    test('should show error when bulk uploading files with missing filenames and allow retry after fixing', async () => {
+      await page.goto(uploadsOne.create)
+
+      await page.setInputFiles(
+        '.file-field input[type="file"]',
+        path.resolve(dirname, './image.png'),
+      )
+      const filename = page.locator('.file-field__filename')
+      await expect(filename).toHaveValue('image.png')
+
+      const bulkUploadButton = page.locator('#field-hasManyUpload button', {
+        hasText: exactText('Create New'),
+      })
+      await bulkUploadButton.click()
+
+      const bulkUploadModal = page.locator('#hasManyUpload-bulk-upload-drawer-slug-1')
+      await expect(bulkUploadModal).toBeVisible()
+
+      await bulkUploadModal
+        .locator('.dropzone input[type="file"]')
+        .setInputFiles([
+          path.resolve(dirname, './image.png'),
+          path.resolve(dirname, './test-image.png'),
+        ])
+
+      await bulkUploadModal
+        .locator('.bulk-upload--file-manager .render-fields #field-prefix')
+        .fill('prefix-one')
+
+      // Clear the filename from the first file
+      await bulkUploadModal.locator('.file-field__filename').clear()
+
+      const nextImageChevronButton = bulkUploadModal.locator(
+        '.bulk-upload--actions-bar__controls button:nth-of-type(2)',
+      )
+      await nextImageChevronButton.click()
+
+      await bulkUploadModal
+        .locator('.bulk-upload--file-manager .render-fields #field-prefix')
+        .fill('prefix-two')
+
+      const saveButton = bulkUploadModal.locator('.bulk-upload--actions-bar__saveButtons button')
+      await saveButton.click()
+
+      // Should show error message for failed files
+      await expect(page.locator('.payload-toast-container')).toContainText('Failed to save 1 files')
+      await expect(page.locator('.payload-toast-container')).toContainText(
+        'Successfully saved 1 files',
+      )
+
+      const errorCount = bulkUploadModal.locator('.file-selections .error-pill__count').first()
+      await expect(errorCount).toHaveText('1')
+
+      await expect(bulkUploadModal).toBeVisible()
+
+      // Navigate back to first file to fix it
+      const prevImageChevronButton = bulkUploadModal.locator(
+        '.bulk-upload--actions-bar__controls button:nth-of-type(1)',
+      )
+      await prevImageChevronButton.click()
+
+      // Should show "A file name is required" error message
+      await expect(bulkUploadModal.locator('.field-error')).toContainText('A file name is required')
+
+      // Filename field should be empty (as we cleared it)
+      await expect(bulkUploadModal.locator('.file-field__filename')).toHaveValue('')
+
+      // Add the filename back
+      await bulkUploadModal.locator('.file-field__filename').fill('fixed-filename.png')
+
+      await saveButton.click()
+
+      await expect(page.locator('.payload-toast-container')).toContainText(
+        'Successfully saved 1 files',
+      )
+
+      await expect(bulkUploadModal).toBeHidden()
+
+      const items = page.locator('#field-hasManyUpload .upload--has-many__dragItem')
+      await expect(items).toHaveCount(2)
+
+      await saveDocAndAssert(page)
+    })
   })
 
   describe('remote url fetching', () => {

test/uploads/payload-types.ts

@@ -3787,6 +3787,6 @@ export interface Auth {
 
 
 declare module 'payload' {
-  // @ts-ignore
+  // @ts-ignore 
   export interface GeneratedTypes extends Config {}
-}
+}