From 26438fa034e1ca03b11b92a867476e104a923365 Mon Sep 17 00:00:00 2001
From: Matthew Penner <me@matthewp.io>
Date: Sat, 24 Jul 2021 11:32:31 -0600
Subject: [PATCH] ui(server/files): add validation for duplicate directory
 names

---
 .../components/dashboard/SSHKeyContainer.tsx     |  5 +++++
 resources/scripts/components/elements/Input.tsx  |  6 ++----
 .../server/files/NewDirectoryButton.tsx          | 16 ++++++++++------
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/resources/scripts/components/dashboard/SSHKeyContainer.tsx b/resources/scripts/components/dashboard/SSHKeyContainer.tsx
index b40e5516d..de0feca06 100644
--- a/resources/scripts/components/dashboard/SSHKeyContainer.tsx
+++ b/resources/scripts/components/dashboard/SSHKeyContainer.tsx
@@ -18,6 +18,7 @@ import useFlash from '@/plugins/useFlash';
 import ConfirmationModal from '@/components/elements/ConfirmationModal';
 import FormikFieldWrapper from '@/components/elements/FormikFieldWrapper';
 import { Textarea } from '@/components/elements/Input';
+import { format } from 'date-fns';
 
 interface Values {
     name: string;
@@ -145,6 +146,10 @@ export default () => {
                                 <FontAwesomeIcon icon={faKey} css={tw`text-neutral-300`}/>
                                 <div css={tw`ml-4 flex-1 overflow-hidden`}>
                                     <p css={tw`text-sm break-words`}>{key.name}</p>
+                                    <p css={tw`text-2xs text-neutral-300 uppercase`}>
+                                        Added at:&nbsp;
+                                        {key.createdAt ? format(key.createdAt, 'MMM do, yyyy HH:mm') : 'Never'}
+                                    </p>
                                 </div>
                                 <button css={tw`ml-4 p-2 text-sm`} onClick={() => setDeleteId(key.id)}>
                                     <FontAwesomeIcon
diff --git a/resources/scripts/components/elements/Input.tsx b/resources/scripts/components/elements/Input.tsx
index 7b76371da..7e7f82942 100644
--- a/resources/scripts/components/elements/Input.tsx
+++ b/resources/scripts/components/elements/Input.tsx
@@ -58,11 +58,9 @@ const inputStyle = css<Props>`
         ${tw`opacity-75`};
     }
 
-    &:not(.ignoreReadOnly):read-only {
+    ${props => props.isLight ? light : css`&:not(.ignoreReadOnly):read-only {
         ${tw`border-neutral-800 bg-neutral-900`};
-    }
-
-    ${props => props.isLight && light};
+    }`};
     ${props => props.hasError && tw`text-red-100 border-red-400 hover:border-red-300`};
 `;
 
diff --git a/resources/scripts/components/server/files/NewDirectoryButton.tsx b/resources/scripts/components/server/files/NewDirectoryButton.tsx
index f6c548337..a9162ec55 100644
--- a/resources/scripts/components/server/files/NewDirectoryButton.tsx
+++ b/resources/scripts/components/server/files/NewDirectoryButton.tsx
@@ -18,10 +18,6 @@ interface Values {
     directoryName: string;
 }
 
-const schema = object().shape({
-    directoryName: string().required('A valid directory name must be provided.'),
-});
-
 const generateDirectoryData = (name: string): FileObject => ({
     key: `dir_${name.split('/', 1)[0] ?? name}`,
     name: name.replace(/^(\/*)/, '').split('/', 1)[0] ?? name,
@@ -42,7 +38,7 @@ export default ({ className }: WithClassname) => {
     const { clearFlashes, clearAndAddHttpError } = useFlash();
     const [ visible, setVisible ] = useState(false);
 
-    const { mutate } = useFileManagerSwr();
+    const { data, mutate } = useFileManagerSwr();
     const directory = ServerContext.useStoreState(state => state.files.directory);
 
     useEffect(() => {
@@ -68,8 +64,16 @@ export default ({ className }: WithClassname) => {
         <>
             <Formik
                 onSubmit={submit}
-                validationSchema={schema}
                 initialValues={{ directoryName: '' }}
+                validationSchema={object().shape({
+                    directoryName: string()
+                        .required('A valid directory name must be provided.')
+                        .test('unique', 'Directory with that name already exists.', v => {
+                            return v !== undefined &&
+                                data !== undefined &&
+                                data.filter(f => f.name.toLowerCase() === v.toLowerCase()).length < 1;
+                        }),
+                })}
             >
                 {({ resetForm, isSubmitting, values }) => (
                     <Modal