fix: Expense & FloatingActions.

client/src/containers/Expenses/ExpenseFloatingActions.js

@@ -23,13 +23,11 @@ export default function ExpenseFloatingFooter({
   isSubmitting,
   onSubmitClick,
   onCancelClick,
-  onDraftClick,
-  onClearClick,
-  onSubmitForm,
-  onResetForm,
   expense,
   expensePublished,
 }) {
+  const { submitForm, resetForm } = useFormikContext();
+
   const handleSubmitPublishBtnClick = (event) => {
     saveInvoke(onSubmitClick, event, {
       redirect: true,
@@ -38,7 +36,7 @@ export default function ExpenseFloatingFooter({
   };
 
   const handleSubmitPublishAndNewBtnClick = (event) => {
-    onSubmitForm();
+    submitForm();
     saveInvoke(onSubmitClick, event, {
       redirect: false,
       publish: true,
@@ -47,7 +45,7 @@ export default function ExpenseFloatingFooter({
   };
 
   const handleSubmitPublishContinueEditingBtnClick = (event) => {
-    onSubmitForm();
+    submitForm();
     saveInvoke(onSubmitClick, event, {
       redirect: false,
       publish: true,
@@ -62,7 +60,7 @@ export default function ExpenseFloatingFooter({
   };
 
   const handleSubmitDraftAndNewBtnClick = (event) => {
-    onSubmitForm();
+    submitForm();
     saveInvoke(onSubmitClick, event, {
       redirect: false,
       publish: false,
@@ -71,7 +69,7 @@ export default function ExpenseFloatingFooter({
   };
 
   const handleSubmitDraftContinueEditingBtnClick = (event) => {
-    onSubmitForm();
+    submitForm();
     saveInvoke(onSubmitClick, event, {
       redirect: false,
       publish: false,
@@ -84,9 +82,8 @@ export default function ExpenseFloatingFooter({
 
   const handleClearBtnClick = (event) => {
     // saveInvoke(onClearClick, event);
-    onResetForm();
+    resetForm();
   };
-
   return (
     <div className={classNames(CLASSES.PAGE_FORM_FLOATING_ACTIONS)}>
       {/* ----------- Save And Publish ----------- */}

client/src/containers/Expenses/ExpenseForm.js

@@ -1,13 +1,11 @@
-import React, {
+import React, { useMemo, useEffect,useState,useCallback } from 'react';
-  useMemo,
-  useEffect,
-} from 'react';
 import { Intent } from '@blueprintjs/core';
 import { useIntl } from 'react-intl';
 import { defaultTo, pick } from 'lodash';
 import { Formik, Form } from 'formik';
 import moment from 'moment';
 import classNames from 'classnames';
+import { useHistory } from 'react-router-dom';
 import { CLASSES } from 'common/classes';
 
 import ExpenseFormHeader from './ExpenseFormHeader';
@@ -28,9 +26,7 @@ import {
   CreateExpenseFormSchema,
   EditExpenseFormSchema,
 } from './ExpenseForm.schema';
-import {
+import { transformErrors } from './utils';
-  transformErrors,
-} from './utils';
 import { compose, repeatValue, orderingLinesIndexes } from 'utils';
 
 const MIN_LINES_NUMBER = 4;
@@ -49,6 +45,7 @@ const defaultInitialValues = {
   description: '',
   reference_no: '',
   currency_code: '',
+  is_published:'',
   categories: [...repeatValue(defaultCategory, MIN_LINES_NUMBER)],
 };
 
@@ -82,7 +79,9 @@ function ExpenseForm({
   onCancelForm,
 }) {
   const isNewMode = !expenseId;
+  const [submitPayload, setSubmitPayload] = useState({});
   const { formatMessage } = useIntl();
+  const history = useHistory();
 
   const validationSchema = isNewMode
     ? CreateExpenseFormSchema
@@ -115,9 +114,7 @@ function ExpenseForm({
           }
         : {
             ...defaultInitialValues,
-            categories: orderingLinesIndexes(
+            categories: orderingLinesIndexes(defaultInitialValues.categories),
-              defaultInitialValues.categories,
-            ),
           }),
     }),
     [expense, baseCurrency, preferredPaymentAccount],
@@ -146,22 +143,30 @@ function ExpenseForm({
 
     const form = {
       ...values,
-      publish: 1,
+      is_published: submitPayload.publish,
       categories,
     };
     // Handle request success.
     const handleSuccess = (response) => {
       AppToaster.show({
         message: formatMessage(
-          { id: isNewMode ?
+          {
-            'the_expense_has_been_successfully_created' : 
+            id: isNewMode
-            'the_expense_has_been_successfully_edited' },
+              ? 'the_expense_has_been_successfully_created'
+              : 'the_expense_has_been_successfully_edited',
+          },
           { number: values.payment_account_id },
         ),
         intent: Intent.SUCCESS,
       });
       setSubmitting(false);
-      resetForm();
+
+      if (submitPayload.redirect) {
+        history.push('/expenses');
+      }
+      if (submitPayload.resetForm) {
+        resetForm();
+      }
     };
 
     // Handle request error
@@ -172,16 +177,30 @@ function ExpenseForm({
     if (isNewMode) {
       requestSubmitExpense(form).then(handleSuccess).catch(handleError);
     } else {
-      requestEditExpense(expense.id, form).then(handleSuccess).catch(handleError);
+      requestEditExpense(expense.id, form)
+        .then(handleSuccess)
+        .catch(handleError);
     }
   };
+  const handleCancelClick = useCallback(() => {
+    history.goBack();
+  }, [history]);
+
+  const handleSubmitClick = useCallback(
+    (event, payload) => {
+      setSubmitPayload({ ...payload });
+    },
+    [setSubmitPayload],
+  );
 
   return (
-    <div className={classNames(
+    <div
-      CLASSES.PAGE_FORM,
+      className={classNames(
-      CLASSES.PAGE_FORM_STRIP_STYLE,
+        CLASSES.PAGE_FORM,
-      CLASSES.PAGE_FORM_EXPENSE
+        CLASSES.PAGE_FORM_STRIP_STYLE,
-    )}>
+        CLASSES.PAGE_FORM_EXPENSE,
+      )}
+    >
       <Formik
         validationSchema={validationSchema}
         initialValues={initialValues}
@@ -192,7 +211,13 @@ function ExpenseForm({
             <ExpenseFormHeader />
             <ExpenseFormBody />
             <ExpenseFormFooter />
-            <ExpenseFloatingFooter />
+            <ExpenseFloatingFooter
+              isSubmitting={isSubmitting}
+              expense={expenseId}
+              expensePublished={values.is_published}
+              onCancelClick={handleCancelClick}
+              onSubmitClick={handleSubmitClick}
+            />
           </Form>
         )}
       </Formik>
@@ -208,6 +233,9 @@ export default compose(
   withExpenseDetail(),
   withSettings(({ organizationSettings, expenseSettings }) => ({
     baseCurrency: organizationSettings?.baseCurrency,
-    preferredPaymentAccount: parseInt(expenseSettings?.preferredPaymentAccount, 10),
+    preferredPaymentAccount: parseInt(
+      expenseSettings?.preferredPaymentAccount,
+      10,
+    ),
   })),
 )(ExpenseForm);