From c5874613fe11de1e624f4131643d54b1cf1039df Mon Sep 17 00:00:00 2001
From: Ricky Smith <jamesricky@me.com>
Date: Wed, 24 Jul 2024 12:06:53 +0200
Subject: [PATCH] Make `onDrop` and `onRemove` optional as they are not needed
 when `readOnly` is set

---
 packages/admin/admin/src/form/file/FileSelect.tsx           | 6 +++---
 .../src/docs/components/FileSelect/FileSelect.stories.tsx   | 6 ------
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/packages/admin/admin/src/form/file/FileSelect.tsx b/packages/admin/admin/src/form/file/FileSelect.tsx
index 16a5f9b03e..b86860f48b 100644
--- a/packages/admin/admin/src/form/file/FileSelect.tsx
+++ b/packages/admin/admin/src/form/file/FileSelect.tsx
@@ -36,8 +36,8 @@ type ThemeProps = ThemedComponentBaseProps<{
 
 export type FileSelectProps<AdditionalValidFileValues = Record<string, unknown>> = {
     files: FileSelectItem<AdditionalValidFileValues>[];
-    onDrop: DropzoneOptions["onDrop"];
-    onRemove: (file: FileSelectItem<AdditionalValidFileValues>) => void;
+    onDrop?: DropzoneOptions["onDrop"];
+    onRemove?: (file: FileSelectItem<AdditionalValidFileValues>) => void;
     onDownload?: (file: FileSelectItem<AdditionalValidFileValues>) => void;
     disabled?: boolean;
     readOnly?: boolean;
@@ -128,7 +128,7 @@ export const FileSelect = <AdditionalValidFileValues = Record<string, unknown>,>
                                                   onDownload(file);
                                               }
                                     }
-                                    onClickDelete={readOnly ? undefined : () => onRemove(file)}
+                                    onClickDelete={readOnly || !onRemove ? undefined : () => onRemove(file)}
                                     {...slotProps?.fileListItem}
                                 />
                             ))}
diff --git a/storybook/src/docs/components/FileSelect/FileSelect.stories.tsx b/storybook/src/docs/components/FileSelect/FileSelect.stories.tsx
index ed32eb037a..a076d50d81 100644
--- a/storybook/src/docs/components/FileSelect/FileSelect.stories.tsx
+++ b/storybook/src/docs/components/FileSelect/FileSelect.stories.tsx
@@ -61,12 +61,6 @@ storiesOf("stories/components/FileSelect", module)
 
         return (
             <FileSelect
-                onDrop={(acceptedFiles, fileRejections) => {
-                    // Handle what happens with the dropped files
-                }}
-                onRemove={(file) => {
-                    // Handle remove
-                }}
                 onDownload={(file) => {
                     // Handle download
                 }}