From 6b6d052e747c617b94d7cfc036c9da196d74d812 Mon Sep 17 00:00:00 2001
From: Patrick Fic <>
Date: Mon, 28 Sep 2020 15:23:28 -0700
Subject: [PATCH] Added read only invocies. Removed added reducer coe that was
 needed BOD-408

---
 bodyshop_translations.babel                   | 23 +++++++-
 .../bill-detail-edit.container.jsx            | 56 ++++++++++++++-----
 .../bill-form/bill-form.component.jsx         | 17 +++---
 .../bill-form/bill-form.container.jsx         | 25 ++++-----
 .../bill-form/bill-form.lines.component.jsx   | 23 +++++---
 .../bill-line-search-select.component.jsx     |  3 +-
 .../bills-list-table.component.jsx            | 17 ++++--
 client/src/graphql/bill-lines.queries.js      | 10 ++++
 client/src/graphql/bills.queries.js           |  2 +
 .../src/pages/bills/bills.page.component.jsx  | 13 ++++-
 .../redux/application/application.actions.js  |  5 --
 .../redux/application/application.reducer.js  |  3 -
 .../application/application.selectors.js      |  5 --
 .../redux/application/application.types.js    |  1 -
 client/src/translations/en_us/common.json     |  1 +
 client/src/translations/es/common.json        |  1 +
 client/src/translations/fr/common.json        |  1 +
 17 files changed, 144 insertions(+), 62 deletions(-)

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index 5d64fdcc6..d6da40666 100644
--- a/bodyshop_translations.babel
+++ b/bodyshop_translations.babel
@@ -1,4 +1,4 @@
-<babeledit_project version="1.2" be_version="2.7.1">
+<babeledit_project be_version="2.7.1" version="1.2">
     <!--
 
     BabelEdit project file
@@ -1567,6 +1567,27 @@
                                             </translation>
                                         </translations>
                                     </concept_node>
+                                    <concept_node>
+                                        <name>exported</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
                                     <concept_node>
                                         <name>federal_tax_rate</name>
                                         <definition_loaded>false</definition_loaded>
diff --git a/client/src/components/bill-detail-edit/bill-detail-edit.container.jsx b/client/src/components/bill-detail-edit/bill-detail-edit.container.jsx
index 2bede8c5c..cd2d785de 100644
--- a/client/src/components/bill-detail-edit/bill-detail-edit.container.jsx
+++ b/client/src/components/bill-detail-edit/bill-detail-edit.container.jsx
@@ -5,7 +5,10 @@ import queryString from "query-string";
 import React, { useEffect, useState } from "react";
 import { useTranslation } from "react-i18next";
 import { useLocation } from "react-router-dom";
-import { UPDATE_BILL_LINE } from "../../graphql/bill-lines.queries";
+import {
+  INSERT_NEW_BILL_LINES,
+  UPDATE_BILL_LINE,
+} from "../../graphql/bill-lines.queries";
 import { QUERY_BILL_BY_PK, UPDATE_BILL } from "../../graphql/bills.queries";
 import AlertComponent from "../alert/alert.component";
 import BillFormContainer from "../bill-form/bill-form.container";
@@ -18,6 +21,7 @@ export default function BillDetailEditcontainer() {
   const [form] = Form.useForm();
   const [updateLoading, setUpdateLoading] = useState(false);
   const [update_bill] = useMutation(UPDATE_BILL);
+  const [insertBillLine] = useMutation(INSERT_NEW_BILL_LINES);
   const [updateBillLine] = useMutation(UPDATE_BILL_LINE);
 
   const { loading, error, data, refetch } = useQuery(QUERY_BILL_BY_PK, {
@@ -37,17 +41,35 @@ export default function BillDetailEditcontainer() {
 
     billlines.forEach((il) => {
       delete il.__typename;
-      updates.push(
-        updateBillLine({
-          variables: {
-            billLineId: il.id,
-            billLine: {
-              ...il,
-              joblineid: il.joblineid === "noline" ? null : il.joblineid,
+
+      if (il.id) {
+        updates.push(
+          updateBillLine({
+            variables: {
+              billLineId: il.id,
+              billLine: {
+                ...il,
+                joblineid: il.joblineid === "noline" ? null : il.joblineid,
+              },
             },
-          },
-        })
-      );
+          })
+        );
+      } else {
+        //It's a new line, have to insert it.
+        updates.push(
+          insertBillLine({
+            variables: {
+              billLines: [
+                {
+                  ...il,
+                  billid: search.billid,
+                  joblineid: il.joblineid === "noline" ? null : il.joblineid,
+                },
+              ],
+            },
+          })
+        );
+      }
     });
 
     await Promise.all(updates);
@@ -62,6 +84,9 @@ export default function BillDetailEditcontainer() {
 
   if (error) return <AlertComponent message={error.message} type="error" />;
   if (!!!search.billid) return <div>{t("bills.labels.noneselected")}</div>;
+
+  const exported = data && data.bills_by_pk && data.bills_by_pk.exported;
+
   return (
     <LoadingSkeleton loading={loading}>
       <Form
@@ -94,10 +119,15 @@ export default function BillDetailEditcontainer() {
             : {}
         }
       >
-        <Button htmlType="submit" loading={updateLoading} type="primary">
+        <Button
+          htmlType="submit"
+          disabled={exported}
+          loading={updateLoading}
+          type="primary"
+        >
           {t("general.actions.save")}
         </Button>
-        <BillFormContainer form={form} billEdit />
+        <BillFormContainer form={form} billEdit disabled={exported} />
         <JobDocumentsGallery
           jobId={data ? data.bills_by_pk.jobid : null}
           billId={search.billid}
diff --git a/client/src/components/bill-form/bill-form.component.jsx b/client/src/components/bill-form/bill-form.component.jsx
index 1c6055c29..6f4e2a60c 100644
--- a/client/src/components/bill-form/bill-form.component.jsx
+++ b/client/src/components/bill-form/bill-form.component.jsx
@@ -29,6 +29,7 @@ const mapDispatchToProps = (dispatch) => ({});
 
 export function BillFormComponent({
   bodyshop,
+  disabled,
   form,
   vendorAutoCompleteOptions,
   lineData,
@@ -74,7 +75,7 @@ export function BillFormComponent({
           ]}
         >
           <JobSearchSelect
-            disabled={billEdit}
+            disabled={billEdit || disabled}
             onBlur={() => {
               if (form.getFieldValue("jobid") !== null) {
                 loadLines({ variables: { id: form.getFieldValue("jobid") } });
@@ -94,6 +95,7 @@ export function BillFormComponent({
           ]}
         >
           <VendorSearchSelect
+            disabled={disabled}
             options={vendorAutoCompleteOptions}
             onSelect={handleVendorSelect}
           />
@@ -111,7 +113,7 @@ export function BillFormComponent({
             },
           ]}
         >
-          <Input />
+          <Input disabled={disabled} />
         </Form.Item>
         <Form.Item
           label={t("bills.fields.date")}
@@ -123,7 +125,7 @@ export function BillFormComponent({
             },
           ]}
         >
-          <FormDatePicker />
+          <FormDatePicker disabled={disabled} />
         </Form.Item>
         <Form.Item
           label={t("bills.fields.is_credit_memo")}
@@ -142,19 +144,19 @@ export function BillFormComponent({
             },
           ]}
         >
-          <CurrencyInput min={0} />
+          <CurrencyInput min={0} disabled={disabled} />
         </Form.Item>
         <Form.Item
           label={t("bills.fields.federal_tax_rate")}
           name="federal_tax_rate"
         >
-          <CurrencyInput min={0} />
+          <CurrencyInput min={0} disabled={disabled} />
         </Form.Item>
         <Form.Item
           label={t("bills.fields.state_tax_rate")}
           name="state_tax_rate"
         >
-          <CurrencyInput min={0} />
+          <CurrencyInput min={0} disabled={disabled} />
         </Form.Item>
         <Form.Item
           label={t("bills.fields.local_tax_rate")}
@@ -164,7 +166,7 @@ export function BillFormComponent({
         </Form.Item>
 
         <Form.Item label={t("bills.fields.allpartslocation")} name="location">
-          <Select style={{ width: "10rem" }}>
+          <Select style={{ width: "10rem" }} disabled={disabled}>
             {bodyshop.md_parts_locations.map((loc, idx) => (
               <Select.Option key={idx} value={loc}>
                 {loc}
@@ -181,6 +183,7 @@ export function BillFormComponent({
         discount={discount}
         form={form}
         responsibilityCenters={responsibilityCenters}
+        disabled={disabled}
       />
 
       <Form.Item
diff --git a/client/src/components/bill-form/bill-form.container.jsx b/client/src/components/bill-form/bill-form.container.jsx
index 004820d1c..cb733d178 100644
--- a/client/src/components/bill-form/bill-form.container.jsx
+++ b/client/src/components/bill-form/bill-form.container.jsx
@@ -11,7 +11,7 @@ const mapStateToProps = createStructuredSelector({
   bodyshop: selectBodyshop,
 });
 
-export function BillFormContainer({ bodyshop, form, billEdit }) {
+export function BillFormContainer({ bodyshop, form, billEdit, disabled }) {
   const { data: VendorAutoCompleteData } = useQuery(SEARCH_VENDOR_AUTOCOMPLETE);
 
   const [loadLines, { data: lineData }] = useLazyQuery(
@@ -19,18 +19,17 @@ export function BillFormContainer({ bodyshop, form, billEdit }) {
   );
 
   return (
-    <div>
-      <BillFormComponent
-        form={form}
-        billEdit={billEdit}
-        vendorAutoCompleteOptions={
-          VendorAutoCompleteData && VendorAutoCompleteData.vendors
-        }
-        loadLines={loadLines}
-        lineData={lineData ? lineData.joblines : []}
-        responsibilityCenters={bodyshop.md_responsibility_centers || null}
-      />
-    </div>
+    <BillFormComponent
+      disabled={disabled}
+      form={form}
+      billEdit={billEdit}
+      vendorAutoCompleteOptions={
+        VendorAutoCompleteData && VendorAutoCompleteData.vendors
+      }
+      loadLines={loadLines}
+      lineData={lineData ? lineData.joblines : []}
+      responsibilityCenters={bodyshop.md_responsibility_centers || null}
+    />
   );
 }
 export default connect(mapStateToProps, null)(BillFormContainer);
diff --git a/client/src/components/bill-form/bill-form.lines.component.jsx b/client/src/components/bill-form/bill-form.lines.component.jsx
index da0098547..40bfea588 100644
--- a/client/src/components/bill-form/bill-form.lines.component.jsx
+++ b/client/src/components/bill-form/bill-form.lines.component.jsx
@@ -15,6 +15,7 @@ import BillLineSearchSelect from "../bill-line-search-select/bill-line-search-se
 import FormListMoveArrows from "../form-list-move-arrows/form-list-move-arrows.component";
 import LayoutFormRow from "../layout-form-row/layout-form-row.component";
 export default function BillEnterModalLinesComponent({
+  disabled,
   lineData,
   discount,
   form,
@@ -45,6 +46,7 @@ export default function BillEnterModalLinesComponent({
                         ]}
                       >
                         <BillLineSearchSelect
+                          disabled={disabled}
                           options={lineData}
                           onSelect={(value, opt) => {
                             setFieldsValue({
@@ -82,7 +84,7 @@ export default function BillEnterModalLinesComponent({
                           },
                         ]}
                       >
-                        <Input />
+                        <Input disabled={disabled} />
                       </Form.Item>
                       <Form.Item
                         label={t("billlines.fields.quantity")}
@@ -95,7 +97,11 @@ export default function BillEnterModalLinesComponent({
                           },
                         ]}
                       >
-                        <InputNumber precision={0} min={0} />
+                        <InputNumber
+                          precision={0}
+                          min={0}
+                          disabled={disabled}
+                        />
                       </Form.Item>
                       <Form.Item
                         label={t("billlines.fields.actual")}
@@ -110,6 +116,7 @@ export default function BillEnterModalLinesComponent({
                       >
                         <CurrencyInput
                           min={0}
+                          disabled={disabled}
                           onBlur={(e) => {
                             setFieldsValue({
                               billlines: getFieldsValue(
@@ -141,7 +148,7 @@ export default function BillEnterModalLinesComponent({
                           },
                         ]}
                       >
-                        <CurrencyInput min={0} />
+                        <CurrencyInput min={0} disabled={disabled} />
                       </Form.Item>
                       <Form.Item shouldUpdate>
                         {() => {
@@ -172,7 +179,7 @@ export default function BillEnterModalLinesComponent({
                           },
                         ]}
                       >
-                        <Select style={{ width: "150px" }}>
+                        <Select style={{ width: "150px" }} disabled={disabled}>
                           {responsibilityCenters.costs.map((item) => (
                             <Select.Option key={item.name}>
                               {item.name}
@@ -187,7 +194,7 @@ export default function BillEnterModalLinesComponent({
                         valuePropName="checked"
                         name={[field.name, "applicable_taxes", "federal"]}
                       >
-                        <Switch />
+                        <Switch disabled={disabled} />
                       </Form.Item>
                       <Form.Item
                         label={t("billlines.fields.state_tax_applicable")}
@@ -195,7 +202,7 @@ export default function BillEnterModalLinesComponent({
                         valuePropName="checked"
                         name={[field.name, "applicable_taxes", "state"]}
                       >
-                        <Switch />
+                        <Switch disabled={disabled} />
                       </Form.Item>
                       <Form.Item
                         label={t("billlines.fields.local_tax_applicable")}
@@ -203,7 +210,7 @@ export default function BillEnterModalLinesComponent({
                         valuePropName="checked"
                         name={[field.name, "applicable_taxes", "local"]}
                       >
-                        <Switch />
+                        <Switch disabled={disabled} />
                       </Form.Item>
                     </LayoutFormRow>
                     <FormListMoveArrows
@@ -212,6 +219,7 @@ export default function BillEnterModalLinesComponent({
                       total={fields.length}
                     />
                     <DeleteFilled
+                      disabled={disabled}
                       onClick={() => {
                         remove(field.name);
                       }}
@@ -223,6 +231,7 @@ export default function BillEnterModalLinesComponent({
             ))}
             <Form.Item>
               <Button
+                disabled={disabled}
                 onClick={() => {
                   add();
                 }}
diff --git a/client/src/components/bill-line-search-select/bill-line-search-select.component.jsx b/client/src/components/bill-line-search-select/bill-line-search-select.component.jsx
index d1d29b26f..c9a07ef10 100644
--- a/client/src/components/bill-line-search-select/bill-line-search-select.component.jsx
+++ b/client/src/components/bill-line-search-select/bill-line-search-select.component.jsx
@@ -6,7 +6,7 @@ import CurrencyFormatter from "../../utils/CurrencyFormatter";
 //To be used as a form element only.
 const { Option } = Select;
 const BillLineSearchSelect = (
-  { value, onChange, options, onBlur, onSelect },
+  { value, onChange, options, onBlur, onSelect, disabled },
   ref
 ) => {
   const [option, setOption] = useState(value);
@@ -20,6 +20,7 @@ const BillLineSearchSelect = (
 
   return (
     <Select
+      disabled={disabled}
       ref={ref}
       showSearch
       autoFocus
diff --git a/client/src/components/bills-list-table/bills-list-table.component.jsx b/client/src/components/bills-list-table/bills-list-table.component.jsx
index e328e6b42..ae1ec7db2 100644
--- a/client/src/components/bills-list-table/bills-list-table.component.jsx
+++ b/client/src/components/bills-list-table/bills-list-table.component.jsx
@@ -13,7 +13,7 @@ import { DateFormatter } from "../../utils/DateFormatter";
 import { alphaSort } from "../../utils/sorters";
 
 const mapStateToProps = createStructuredSelector({
-  jobRO: selectJobReadOnly,
+  //jobRO: selectJobReadOnly,
 });
 
 const mapDispatchToProps = (dispatch) => ({
@@ -26,7 +26,6 @@ const mapDispatchToProps = (dispatch) => ({
 });
 
 export function BillsListTableComponent({
-  jobRO,
   job,
   billsQuery,
   handleOnRowClick,
@@ -94,13 +93,22 @@ export function BillsListTableComponent({
         state.sortedInfo.order,
       render: (text, record) => <Checkbox checked={record.is_credit_memo} />,
     },
+    {
+      title: t("bills.fields.exported"),
+      dataIndex: "exported",
+      key: "exported",
+      sorter: (a, b) => a.exported - b.exported,
+      sortOrder:
+        state.sortedInfo.columnKey === "exported" && state.sortedInfo.order,
+      render: (text, record) => <Checkbox checked={record.exported} />,
+    },
     {
       title: t("general.labels.actions"),
       dataIndex: "actions",
       key: "actions",
       render: (text, record) => (
         <div>
-          {jobRO ? (
+          {record.exported ? (
             <Button disabled>{t("bills.actions.edit")}</Button>
           ) : (
             <Link
@@ -237,7 +245,7 @@ export function BillsListTableComponent({
           </Descriptions.Item>
         </Descriptions>
         <Button
-          disabled={record.is_credit_memo || jobRO}
+          disabled={record.is_credit_memo || record.exported}
           onClick={() =>
             setPartsOrderContext({
               actions: {},
@@ -302,7 +310,6 @@ export function BillsListTableComponent({
             {job ? (
               <div>
                 <Button
-                  disabled={jobRO}
                   onClick={() => {
                     setBillEnterContext({
                       actions: { refetch: billsQuery.refetch },
diff --git a/client/src/graphql/bill-lines.queries.js b/client/src/graphql/bill-lines.queries.js
index 3242d8195..e0f22b7ee 100644
--- a/client/src/graphql/bill-lines.queries.js
+++ b/client/src/graphql/bill-lines.queries.js
@@ -12,3 +12,13 @@ export const UPDATE_BILL_LINE = gql`
     }
   }
 `;
+
+export const INSERT_NEW_BILL_LINES = gql`
+  mutation INSERT_NEW_BILL_LINES($billLines: [billlines_insert_input!]!) {
+    insert_billlines(objects: $billLines) {
+      returning {
+        id
+      }
+    }
+  }
+`;
diff --git a/client/src/graphql/bills.queries.js b/client/src/graphql/bills.queries.js
index e341dfc74..0356d52f5 100644
--- a/client/src/graphql/bills.queries.js
+++ b/client/src/graphql/bills.queries.js
@@ -35,6 +35,7 @@ export const QUERY_ALL_BILLS_PAGINATED = gql`
       total
       invoice_number
       date
+      exported
       job {
         id
         ro_number
@@ -69,6 +70,7 @@ export const QUERY_BILLS_BY_JOBID = gql`
       }
       order_date
       deliver_by
+      exported
       parts_order_lines {
         id
         act_price
diff --git a/client/src/pages/bills/bills.page.component.jsx b/client/src/pages/bills/bills.page.component.jsx
index c82a0a56c..0728cf806 100644
--- a/client/src/pages/bills/bills.page.component.jsx
+++ b/client/src/pages/bills/bills.page.component.jsx
@@ -80,7 +80,18 @@ export function BillsListPage({
       sortOrder:
         state.sortedInfo.columnKey === "is_credit_memo" &&
         state.sortedInfo.order,
-      render: (text, record) => <Checkbox checked={record.is_credit_memo} />,
+      render: (text, record) => (
+        <Checkbox disabled checked={record.is_credit_memo} />
+      ),
+    },
+    {
+      title: t("bills.fields.exported"),
+      dataIndex: "exported",
+      key: "exported",
+      sorter: (a, b) => a.exported - b.exported,
+      sortOrder:
+        state.sortedInfo.columnKey === "exported" && state.sortedInfo.order,
+      render: (text, record) => <Checkbox disabled checked={record.exported} />,
     },
     {
       title: t("general.labels.actions"),
diff --git a/client/src/redux/application/application.actions.js b/client/src/redux/application/application.actions.js
index b825f15e1..84d63ca37 100644
--- a/client/src/redux/application/application.actions.js
+++ b/client/src/redux/application/application.actions.js
@@ -43,8 +43,3 @@ export const setJobReadOnly = (bool) => ({
   type: ApplicationActionTypes.SET_JOB_READONLY,
   payload: bool,
 });
-
-export const setBillReadOnly = (bool) => ({
-  type: ApplicationActionTypes.SET_BILL_READONLY,
-  payload: bool,
-});
diff --git a/client/src/redux/application/application.reducer.js b/client/src/redux/application/application.reducer.js
index af89b631b..338c45af0 100644
--- a/client/src/redux/application/application.reducer.js
+++ b/client/src/redux/application/application.reducer.js
@@ -11,7 +11,6 @@ const INITIAL_STATE = {
     error: null,
   },
   jobReadOnly: false,
-  billReadOnly: false,
 };
 
 const applicationReducer = (state = INITIAL_STATE, action) => {
@@ -65,8 +64,6 @@ const applicationReducer = (state = INITIAL_STATE, action) => {
         loading: false,
       };
 
-    case ApplicationActionTypes.SET_BILL_READONLY:
-      return { ...state, billReadOnly: action.payload };
     case ApplicationActionTypes.SET_JOB_READONLY:
       return { ...state, jobReadOnly: action.payload };
 
diff --git a/client/src/redux/application/application.selectors.js b/client/src/redux/application/application.selectors.js
index deb4e0abc..bd3090424 100644
--- a/client/src/redux/application/application.selectors.js
+++ b/client/src/redux/application/application.selectors.js
@@ -36,8 +36,3 @@ export const selectJobReadOnly = createSelector(
   [selectApplication],
   (application) => application.jobReadOnly
 );
-
-export const selectBillReadOnly = createSelector(
-  [selectApplication],
-  (application) => application.billReadOnly
-);
diff --git a/client/src/redux/application/application.types.js b/client/src/redux/application/application.types.js
index b2c3a9804..929bb4e76 100644
--- a/client/src/redux/application/application.types.js
+++ b/client/src/redux/application/application.types.js
@@ -8,6 +8,5 @@ const ApplicationActionTypes = {
   ADD_RECENT_ITEM: "ADD_RECENT_ITEM",
   SET_SELECTED_HEADER: "SET_SELECTED_HEADER",
   SET_JOB_READONLY: "SET_JOB_READONLY",
-  SET_BILL_READONLY: "SET_BILL_READONLY",
 };
 export default ApplicationActionTypes;
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index f583ecf10..7bd277630 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -116,6 +116,7 @@
 			"fields": {
 				"allpartslocation": "Parts Bin",
 				"date": "Bill Date",
+				"exported": "Exported",
 				"federal_tax_rate": "Federal Tax Rate",
 				"invoice_number": "Invoice Number",
 				"is_credit_memo": "Credit Memo?",
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index a18751fd0..8109fdc25 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -116,6 +116,7 @@
 			"fields": {
 				"allpartslocation": "",
 				"date": "",
+				"exported": "",
 				"federal_tax_rate": "",
 				"invoice_number": "",
 				"is_credit_memo": "",
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index 5e0322920..cf9e8c16f 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -116,6 +116,7 @@
 			"fields": {
 				"allpartslocation": "",
 				"date": "",
+				"exported": "",
 				"federal_tax_rate": "",
 				"invoice_number": "",
 				"is_credit_memo": "",