feat: Remove 'unique' property from List schema

theopensystemslab · Jun 27, 2024 · 7c5d743 · 7c5d743
1 parent 91784c2
commit 7c5d743
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 15 deletions.
diff --git a/editor.planx.uk/src/@planx/components/List/Public/Fields.tsx b/editor.planx.uk/src/@planx/components/List/Public/Fields.tsx
@@ -154,18 +154,6 @@ export const SelectFieldInput: React.FC<Props<QuestionField>> = (props) => {
   const { formik, activeIndex } = useListContext();
   const { id, data, required } = props;
 
-  const isDisabled = (option: Option) => {
-    if (!props.unique) return false;
-
-    const existingValues = formik.values.userData
-      .map((response) => response[data.fn])
-      .filter(
-        (value) => value === option.data.val || value === option.data.text,
-      );
-
-    return existingValues.includes(option.data.val || option.data.text);
-  };
-
   return (
     <InputLabel
       label={required === false ? data.title + " (optional)" : data.title}
@@ -188,7 +176,6 @@ export const SelectFieldInput: React.FC<Props<QuestionField>> = (props) => {
             <MenuItem
               key={option.id}
               value={option.data.val || option.data.text}
-              disabled={isDisabled(option)}
             >
               {option.data.text}
             </MenuItem>

diff --git a/editor.planx.uk/src/@planx/components/List/Public/index.test.tsx b/editor.planx.uk/src/@planx/components/List/Public/index.test.tsx
@@ -383,7 +383,6 @@ describe("Form validation and error handling", () => {
   test.todo("text fields use existing validation schemas");
   test.todo("number fields use existing validation schemas");
   test.todo("question fields use validation schema");
-  test.todo("unique constraints are enforced on question where this is set");
   test.todo("optional fields can be empty when saving an item");
   test.todo("an error displays if the minimum number of items is not met");
   test.todo("an error displays if the maximum number of items is exceeded");

diff --git a/editor.planx.uk/src/@planx/components/List/model.ts b/editor.planx.uk/src/@planx/components/List/model.ts
@@ -42,7 +42,6 @@ export type NumberField = {
 export type QuestionField = {
   type: "question";
   required?: boolean;
-  unique?: boolean;
   data: QuestionInput & { fn: string };
 };