fix bugs

kingyau991319 · Jun 11, 2022 · 51ce802 · 51ce802
1 parent 5a1a32d
commit 51ce802
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 5 deletions.
diff --git a/src/components/NewResv/Form/PersonInfo/Interface/InputElementType.ts b/src/components/NewResv/Form/PersonInfo/Interface/InputElementType.ts
@@ -1,2 +1,2 @@
 // the type of the input element
-export type InputType = 'name' | 'age' | 'email' | 'phone' | 'remark';
+export type InputType = 'name' | 'age' | 'email' | 'phone' | 'remark' | 'gender';
diff --git a/src/components/NewResv/Form/PersonInfo/Person.tsx b/src/components/NewResv/Form/PersonInfo/Person.tsx
@@ -25,7 +25,7 @@ import Remark from './TextInputElement/Remark';
 import Gender from './TextInputElement/Gender';
 
 // Interface
-import { PersonData, PhoneData } from './Interface/PersonData';
+import { GenderType, PersonData, PhoneData } from './Interface/PersonData';
 
 const Person = 
   (props : {
@@ -70,7 +70,7 @@ const Person =
     changeValue(id, 'phone', inputPhone)
   }
 
-  const setGender = (inputGender : string) => {
+  const setGender = (inputGender : GenderType) => {
     changeValue(id, 'gender', inputGender)
   }
 

diff --git a/src/components/NewResv/Form/PersonInfo/PersonInfo.tsx b/src/components/NewResv/Form/PersonInfo/PersonInfo.tsx
@@ -64,11 +64,14 @@ const PersonInfo = (props : {persons : PersonData[], setPersons : Function}) =>
           case 'phone':
             updatePerson.phone = value;
             break;
+          case 'gender':
+            updatePerson.gender = value;
+            break;
           case 'remark':
             updatePerson.remark = value;
             break;
           default:
-            break;
+            return;
         }
       persons[id-1] = updatePerson;
       setPersons([...persons]);

diff --git a/src/components/NewResv/Form/PersonInfo/TextInputElement/Gender.tsx b/src/components/NewResv/Form/PersonInfo/TextInputElement/Gender.tsx
@@ -30,7 +30,7 @@ const Gender = (props : {value:GenderType, setGender:Function}) => {
       <Select
         onChange={changeGender}
         label={t("gender")}
-        defaultValue={value}
+        defaultValue={'other'}
       >
         <MenuItem value='male'> {t('male')} </MenuItem>
         <MenuItem value='female'> {t('female')} </MenuItem>