Refactors the handleSubmit function

bbc · Sep 9, 2019 · b765718 · b765718
1 parent a3da609
commit b765718
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/packages/components/FormModal/ItemForm/index.js b/packages/components/FormModal/ItemForm/index.js
@@ -13,20 +13,21 @@ const ItemForm = (props) => {
   const [ title, setTitle ] = useState(props.title);
 
   const handleSubmit = (event) => {
+    const form = event.currentTarget;
     event.preventDefault();
     event.stopPropagation();
-
-    if (isValidated) {
+    if (!form.checkValidity()) {
+      setIsValidated(true);
+    } else if (form.checkValidity()) {
+      setIsValidated(true);
       const editedProject = {
         title: title,
         description: description,
         id: props.id
       };
 
       props.handleSaveForm(editedProject);
-    } else if (!isValidated) {
-      setIsValidated(true);
-    };
+    }
   };
 
   return (
@@ -80,7 +81,7 @@ ItemForm.propTypes = {
   id: PropTypes.number.isRequired,
   title: PropTypes.string,
   description: PropTypes.string,
-  showModal: PropTypes.bool.isRequired,
+  isNewItemModalShow: PropTypes.bool.isRequired,
   modalTitle: PropTypes.string.isRequired,
   handleSaveForm: PropTypes.func.isRequired,
 };