WIP: allocate landed cost.

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostDialogContent.js

@@ -8,10 +8,10 @@ import AllocateLandedCostForm from './AllocateLandedCostForm';
 export default function AllocateLandedCostDialogContent({
   // #ownProps
   dialogName,
-  bill,
+  billId,
 }) {
   return (
-    <AllocateLandedCostDialogProvider billId={bill} dialogName={dialogName}>
+    <AllocateLandedCostDialogProvider billId={billId} dialogName={dialogName}>
       <AllocateLandedCostForm />
     </AllocateLandedCostDialogProvider>
   );

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostDialogProvider.js

@@ -2,9 +2,6 @@ import React from 'react';
 import { DialogContent } from 'components';
 import { useBill, useCreateLandedCost } from 'hooks/query';
 
-import { map, omit, pick } from 'lodash';
-import * as R from 'ramda';
-
 const AllocateLandedCostDialogContext = React.createContext();
 
 /**
@@ -24,21 +21,10 @@ function AllocateLandedCostDialogProvider({
   // Create landed cost mutations.
   const { mutateAsync: createLandedCostMutate } = useCreateLandedCost();
 
-  // const L = [bill].map(({ entries: items }) => ({
-  //   items,
-  // }));
-  // let obj = { oldKey: 1, b: 2, c: 3 };
-  // const { oldKey: newKey, ...rest } = obj;
-  // obj = { newKey, ...rest };
-
-  // const obj = { ...pick(bill, ['entries']).map((index) => index) };
-
-
   // provider payload.
   const provider = {
-    items: {
+    isBillLoading,
-      ...pick(bill, ['entries']),
+    bill,
-    },
     dialogName,
     query,
     createLandedCostMutate,

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostEntriesTable.js

@@ -10,8 +10,7 @@ export default function AllocateLandedCostEntriesTable({
   onUpdateData,
   entries,
 }) {
-
+  // Allocate landed cost entries table columns.
-  // allocate landed cost entries table columns.
   const columns = React.useMemo(
     () => [
       {

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostFloatingActions.js

@@ -23,14 +23,14 @@ function AllocateLandedCostFloatingActions({
   return (
     <div className={Classes.DIALOG_FOOTER}>
       <div className={Classes.DIALOG_FOOTER_ACTIONS}>
-        <Button onClick={handleCancelBtnClick} style={{ minWidth: '75px' }}>
+        <Button onClick={handleCancelBtnClick} style={{ minWidth: '85px' }}>
           <T id={'cancel'} />
         </Button>
         <Button
           intent={Intent.PRIMARY}
-          disabled={isSubmitting}
+          style={{ minWidth: '85px' }}
-          style={{ minWidth: '75px' }}
           type="submit"
+          loading={isSubmitting}
         >
           {<T id={'save'} />}
         </Button>

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostForm.js

@@ -3,7 +3,6 @@ import { Formik } from 'formik';
 import { Intent } from '@blueprintjs/core';
 import intl from 'react-intl-universal';
 import moment from 'moment';
-import { pick, omit } from 'lodash';
 
 import 'style/pages/AllocateLandedCost/AllocateLandedCostForm.scss';
 
@@ -12,8 +11,9 @@ import { AllocateLandedCostFormSchema } from './AllocateLandedCostForm.schema';
 import { useAllocateLandedConstDialogContext } from './AllocateLandedCostDialogProvider';
 import AllocateLandedCostFormContent from './AllocateLandedCostFormContent';
 import withDialogActions from 'containers/Dialog/withDialogActions';
-import { compose } from 'utils';
+import { compose, transformToForm } from 'utils';
 
+// Default form initial values.
 const defaultInitialValues = {
   transaction_type: 'Bill',
   transaction_date: moment(new Date()).format('YYYY-MM-DD'),
@@ -21,10 +21,12 @@ const defaultInitialValues = {
   transaction_entry_id: '',
   amount: '',
   allocation_method: 'quantity',
-  items: {
+  items: [
+    {
       entry_id: '',
       cost: '',
     },
+  ],
 };
 
 /**
@@ -34,32 +36,36 @@ function AllocateLandedCostForm({
   // #withDialogActions
   closeDialog,
 }) {
-  const { items, dialogName, createLandedCostMutate } =
+  const { dialogName, bill, billId, createLandedCostMutate } =
     useAllocateLandedConstDialogContext();
 
   // Initial form values.
   const initialValues = {
     ...defaultInitialValues,
-    ...items,
+    items: bill.entries.map((entry) => ({
+      ...entry,
+      entry_id: entry.id,
+      cost: '',
+    })),
   };
 
   // Handle form submit.
   const handleFormSubmit = (values, { setSubmitting }) => {
     setSubmitting(false);
-    closeDialog(dialogName);
 
-    const entries = [values]
+    // Filters the entries has no cost.
-      .filter((entry) => entry.id && entry.cost)
+    const entries = values.items
-      .map((entry) => ({
+      .filter((entry) => entry.entry_id && entry.cost)
-        entry_id: entry.id,
+      .map((entry) => transformToForm(entry, defaultInitialValues.items[0]));
-        ...pick(entry, ['cost']),
-      }));
 
+    if (entries.length <= 0) {
+      AppToaster.show({ message: 'Something wrong!', intent: Intent.DANGER });
+      return;
+    }
     const form = {
       ...values,
-      // items:{entries},
+      items: entries,
     };
-
     // Handle the request success.
     const onSuccess = (response) => {
       AppToaster.show({
@@ -67,17 +73,15 @@ function AllocateLandedCostForm({
         intent: Intent.SUCCESS,
       });
       setSubmitting(false);
+      closeDialog(dialogName);
     };
 
     // Handle the request error.
-    const onError = ({
+    const onError = () => {
-      response: {
-        data: { errors },
-      },
-    }) => {
       setSubmitting(false);
+      AppToaster.show({ message: 'Something went wrong!', intent: Intent.DANGER });
     };
-    createLandedCostMutate(form).then(onSuccess).catch(onError);
+    createLandedCostMutate([billId, form]).then(onSuccess).catch(onError);
   };
 
   return (

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostForm.schema.js

@@ -8,7 +8,7 @@ const Schema = Yup.object().shape({
   transaction_entry_id: Yup.string().label(intl.get('transaction_line')),
   amount: Yup.number().label(intl.get('amount')),
   allocation_method: Yup.string().trim(),
-  entries: Yup.array().of(
+  items: Yup.array().of(
     Yup.object().shape({
       entry_id: Yup.number().nullable(),
       cost: Yup.number().nullable(),

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostFormBody.js

@@ -7,20 +7,18 @@ import AllocateLandedCostEntriesTable from './AllocateLandedCostEntriesTable';
 export default function AllocateLandedCostFormBody() {
   return (
     <div className={classNames(CLASSES.PAGE_FORM_BODY)}>
-      <FastField name={'entries'}>
+      <FastField name={'items'}>
         {({
           form: { setFieldValue, values },
           field: { value },
           meta: { error, touched },
         }) => (
-          <>
           <AllocateLandedCostEntriesTable
             entries={value}
             onUpdateData={(newEntries) => {
-                setFieldValue('entries', newEntries);
+              setFieldValue('items', newEntries);
             }}
           />
-          </>
         )}
       </FastField>
     </div>

client/src/containers/Dialogs/AllocateLandedCostDialog/AllocateLandedCostFormFields.js

@@ -8,7 +8,7 @@ import {
   InputGroup,
 } from '@blueprintjs/core';
 import classNames from 'classnames';
-import { FormattedMessage as T } from 'components';
+import { FormattedMessage as T, If } from 'components';
 import intl from 'react-intl-universal';
 import { inputIntent, handleStringChange } from 'utils';
 import { FieldRequiredHint, ListSelect } from 'components';
@@ -29,10 +29,11 @@ export default function AllocateLandedCostFormFields() {
     data: { transactions },
   } = useLandedCostTransaction(values.transaction_type);
 
-  const transactionEntry = getEntriesByTransactionId(
+  // Retrieve entries of the given transaction id.
+  const transactionEntries = React.useMemo(() => getEntriesByTransactionId(
     transactions,
     values.transaction_id,
-  );
+  ), [transactions, values.transaction_id]);
 
   return (
     <div className={Classes.DIALOG_BODY}>
@@ -71,7 +72,7 @@ export default function AllocateLandedCostFormFields() {
         {({ form, field: { value }, meta: { error, touched } }) => (
           <FormGroup
             label={<T id={'transaction_id'} />}
-            // labelInfo={<FieldRequiredHint />}
+            labelInfo={<FieldRequiredHint />}
             intent={inputIntent({ error, touched })}
             helperText={<ErrorMessage name="transaction_id" />}
             className={classNames(CLASSES.FILL, 'form-group--transaction_id')}
@@ -95,6 +96,7 @@ export default function AllocateLandedCostFormFields() {
       </Field>
 
       {/*------------ Transaction line  -----------*/}
+      <If condition={transactionEntries.length > 0}>
         <Field name={'transaction_entry_id'}>
           {({ form, field: { value }, meta: { error, touched } }) => (
             <FormGroup
@@ -108,8 +110,9 @@ export default function AllocateLandedCostFormFields() {
               inline={true}
             >
               <ListSelect
-              items={transactionEntry}
+                items={transactionEntries}
-              onItemSelect={({ id }) => {
+                onItemSelect={({ id, amount }) => {
+                  form.setFieldValue('amount', amount)
                   form.setFieldValue('transaction_entry_id', id);
                 }}
                 filterable={false}
@@ -123,6 +126,7 @@ export default function AllocateLandedCostFormFields() {
             </FormGroup>
           )}
         </Field>
+      </If>
 
       {/*------------ Amount -----------*/}
       <FastField name={'amount'}>

client/src/containers/Dialogs/AllocateLandedCostDialog/index.js

@@ -25,7 +25,7 @@ function AllocateLandedCostDialog({
     >
       <DialogSuspense>
         <AllocateLandedCostDialogContent
-          bill={payload.billId}
+          billId={payload.billId}
           dialogName={dialogName}
         />
       </DialogSuspense>

client/src/containers/Dialogs/AllocateLandedCostDialog/utils.js

@@ -1,3 +1,6 @@
+/**
+ * Retrieve transaction entries of the given transaction id.
+ */
 export function getEntriesByTransactionId(transactions, id) {
   const transaction = transactions.find((trans) => trans.id === id);
   return transaction ? transaction.entries : [];

client/src/hooks/query/landedCost.js

@@ -21,7 +21,8 @@ export function useCreateLandedCost(props) {
   const apiRequest = useApiRequest();
 
   return useMutation(
-    (id) => apiRequest.post(`purchases/landed-cost/bills/${id}/allocate`),
+    ([id, values]) =>
+      apiRequest.post(`purchases/landed-cost/bills/${id}/allocate`, values),
     {
       onSuccess: (res, id) => {
         // Common invalidate queries.

client/src/lang/en/index.js

@@ -1067,6 +1067,7 @@ export default {
   cash_flow_statement: 'Cash Flow Statement',
   statement_of_cash_flow: 'Statement of Cash Flow ',
   inventory_item_details: 'Inventory Item Details',
-  congratulations: 'Congratulations'
+  congratulations: 'Congratulations',
+  "all_items"
 };
 

client/src/style/pages/AllocateLandedCost/AllocateLandedCostForm.scss

@@ -3,9 +3,12 @@
   width: 700px;
 
   .bp3-dialog-body {
+    .bp3-form-group{
+      margin-bottom: 18px;
+    }
     .bp3-form-group.bp3-inline {
       .bp3-label {
-        min-width: 140px;
+        min-width: 150px;
       }
       .bp3-form-content {
         width: 300px;
@@ -17,6 +20,10 @@
     }
   }
 
+  .bp3-dialog-footer{
+    padding-top: 10px;
+  }
+
   .bigcapital-datatable {
     .table {
       // max-height: 300px;
@@ -37,6 +44,19 @@
           margin-left: -1px;
           border-left: 1px solid #ececec;
         }
+
+        .bp3-form-group{
+          margin-bottom: 0;
+
+          &:not(.bp3-intent-danger) .bp3-input{
+            border: 1px solid #d0dfe2;
+
+            &:focus{
+              box-shadow: 0 0 0 1px #116cd0;
+              border-color: #116cd0;
+            }
+          }
+        }
       }
     }
   }