From 415301d9d18ee965fd706d88e3715c98f87fa871 Mon Sep 17 00:00:00 2001
From: "Michael B. Klein" <mbklein@gmail.com>
Date: Mon, 30 Sep 2024 20:30:25 +0000
Subject: [PATCH] Fix "files is not defined" error in S3 object picker

---
 .../js/components/Work/Tabs/Preservation/S3ObjectPicker.jsx   | 2 +-
 .../js/components/Work/Tabs/Preservation/S3ObjectProvider.jsx | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/assets/js/components/Work/Tabs/Preservation/S3ObjectPicker.jsx b/app/assets/js/components/Work/Tabs/Preservation/S3ObjectPicker.jsx
index dc1d94c80..641960a7b 100644
--- a/app/assets/js/components/Work/Tabs/Preservation/S3ObjectPicker.jsx
+++ b/app/assets/js/components/Work/Tabs/Preservation/S3ObjectPicker.jsx
@@ -47,7 +47,7 @@ const S3ObjectPicker = ({
       case ChonkyActions.ChangeSelection.id:
         if (
           action.payload.selection.size == 0 &&
-          files.find(({ id }) => selectedFile == id)
+          files?.find(({ id }) => selectedFile == id)
         ) {
           fileBrowserRef.current.setFileSelection(new Set([selectedFile]));
           return;
diff --git a/app/assets/js/components/Work/Tabs/Preservation/S3ObjectProvider.jsx b/app/assets/js/components/Work/Tabs/Preservation/S3ObjectProvider.jsx
index 17f581f3b..db2775e41 100644
--- a/app/assets/js/components/Work/Tabs/Preservation/S3ObjectProvider.jsx
+++ b/app/assets/js/components/Work/Tabs/Preservation/S3ObjectProvider.jsx
@@ -39,9 +39,9 @@ const S3ObjectProvider = forwardRef(
     useImperativeHandle(ref, () => ({
       findFileSetByUri: (value) => {
         const objects = data?.ListIngestBucketObjects?.objects;
-        if (!objects) return null;
+        const found = objects?.find(({ uri }) => uri == value);
+        if (!found) return null;
 
-        const found = objects.find(({ uri }) => uri == value);
         return { ...found, key: found.uri, uri: undefined };
       },
     }));