From 9533e7c4e07885eec1ae5699f0039fb109d841b0 Mon Sep 17 00:00:00 2001
From: Patrick Fic <snaptsoft@gmail.com>
Date: Tue, 16 Feb 2021 17:09:01 -0800
Subject: [PATCH] WIP Exporting Fixes

---
 bodyshop_translations.babel                   | 23 +++++-
 .../accounting-payables-table.component.jsx   |  2 +-
 .../jobs-export-all-button.component.jsx      | 78 ++++++++++---------
 .../payment-export-button.component.jsx       | 19 ++---
 .../payment-form/payment-form.component.jsx   |  8 +-
 .../payments-export-all-button.component.jsx  | 78 ++++++++++---------
 client/src/translations/en_us/common.json     |  1 +
 client/src/translations/es/common.json        |  1 +
 client/src/translations/fr/common.json        |  1 +
 server/accounting/qbxml/qbxml-payables.js     |  3 +-
 server/accounting/qbxml/qbxml-payments.js     | 72 +++++++++--------
 server/accounting/qbxml/qbxml-receivables.js  |  7 +-
 server/graphql-client/queries.js              |  7 +-
 13 files changed, 174 insertions(+), 126 deletions(-)

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index c1c2795e0..58d320119 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
@@ -23689,6 +23689,27 @@
                             <folder_node>
                                 <name>successes</name>
                                 <children>
+                                    <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>payment</name>
                                         <definition_loaded>false</definition_loaded>
diff --git a/client/src/components/accounting-payables-table/accounting-payables-table.component.jsx b/client/src/components/accounting-payables-table/accounting-payables-table.component.jsx
index b60ab0f3f..a7663466a 100644
--- a/client/src/components/accounting-payables-table/accounting-payables-table.component.jsx
+++ b/client/src/components/accounting-payables-table/accounting-payables-table.component.jsx
@@ -152,7 +152,7 @@ export default function AccountingPayablesTableComponent({ loading, bills }) {
           return (
             <div className="imex-table-header">
               <PayableExportAll
-                billIds={selectedBills}
+                billids={selectedBills}
                 disabled={transInProgress || selectedBills.length === 0}
                 loadingCallback={setTransInProgress}
                 completedCallback={setSelectedBills}
diff --git a/client/src/components/jobs-export-all-button/jobs-export-all-button.component.jsx b/client/src/components/jobs-export-all-button/jobs-export-all-button.component.jsx
index 48b2761c5..def06cad1 100644
--- a/client/src/components/jobs-export-all-button/jobs-export-all-button.component.jsx
+++ b/client/src/components/jobs-export-all-button/jobs-export-all-button.component.jsx
@@ -1,6 +1,7 @@
 import { useMutation } from "@apollo/react-hooks";
 import { Button, notification } from "antd";
 import axios from "axios";
+import _ from "lodash";
 import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { connect } from "react-redux";
@@ -71,45 +72,50 @@ export function JobsExportAllButton({
     }
 
     console.log("PartnerResponse", PartnerResponse);
+    const groupedData = _.groupBy(PartnerResponse.data, "id");
+    const proms = [];
 
-    //Check to see if any of them failed. If they didn't don't execute the update.
-    const failedTransactions = PartnerResponse.data.filter((r) => !r.success);
-    const successfulTransactions = PartnerResponse.data.filter(
-      (r) => r.success
-    );
-    if (failedTransactions.length > 0) {
-      //Uh oh. At least one was no good.
-      failedTransactions.map((ft) =>
-        notification["error"]({
-          message: t("jobs.errors.exporting", {
-            error: ft.errorMessage || "",
-          }),
-        })
-      );
-    }
-    if (successfulTransactions.length > 0) {
-      const jobUpdateResponse = await updateJob({
-        variables: {
-          jobIds: successfulTransactions.map((st) => st.id),
-          job: {
-            status: bodyshop.md_ro_statuses.default_exported || "Exported*",
-            date_exported: new Date(),
-          },
-        },
+    Object.keys(groupedData).forEach((key) => {
+      proms.push(async () => {
+        //Check to see if any of them failed. If they didn't don't execute the update.
+        const failedTransactions = groupedData[key].filter((r) => !r.success);
+
+        if (failedTransactions.length > 0) {
+          //Uh oh. At least one was no good.
+          failedTransactions.map((ft) =>
+            notification["error"]({
+              message: t("jobs.errors.exporting", {
+                error: ft.errorMessage || "",
+              }),
+            })
+          );
+        } else {
+          const jobUpdateResponse = await updateJob({
+            variables: {
+              jobIds: [key],
+              job: {
+                status: bodyshop.md_ro_statuses.default_exported || "Exported*",
+                date_exported: new Date(),
+              },
+            },
+          });
+
+          if (!!!jobUpdateResponse.errors) {
+            notification["success"]({
+              message: t("jobs.successes.exported"),
+            });
+          } else {
+            notification["error"]({
+              message: t("jobs.errors.exporting", {
+                error: JSON.stringify(jobUpdateResponse.error),
+              }),
+            });
+          }
+        }
       });
+    });
+    await Promise.all(proms);
 
-      if (!!!jobUpdateResponse.errors) {
-        notification["success"]({
-          message: t("jobs.successes.exported"),
-        });
-      } else {
-        notification["error"]({
-          message: t("jobs.errors.exporting", {
-            error: JSON.stringify(jobUpdateResponse.error),
-          }),
-        });
-      }
-    }
     if (!!completedCallback) completedCallback([]);
     if (!!loadingCallback) loadingCallback(false);
     setLoading(false);
diff --git a/client/src/components/payment-export-button/payment-export-button.component.jsx b/client/src/components/payment-export-button/payment-export-button.component.jsx
index 3a54cbad3..84082cad3 100644
--- a/client/src/components/payment-export-button/payment-export-button.component.jsx
+++ b/client/src/components/payment-export-button/payment-export-button.component.jsx
@@ -5,10 +5,9 @@ import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { connect } from "react-redux";
 import { createStructuredSelector } from "reselect";
-import { auth } from "../../firebase/firebase.utils";
+import { auth, logImEXEvent } from "../../firebase/firebase.utils";
 import { UPDATE_PAYMENTS } from "../../graphql/payments.queries";
 import { selectBodyshop } from "../../redux/user/user.selectors";
-import { logImEXEvent } from "../../firebase/firebase.utils";
 
 const mapStateToProps = createStructuredSelector({
   bodyshop: selectBodyshop,
@@ -74,9 +73,7 @@ export function PaymentExportButton({
 
     console.log("handleQbxml -> PartnerResponse", PartnerResponse);
     const failedTransactions = PartnerResponse.data.filter((r) => !r.success);
-    const successfulTransactions = PartnerResponse.data.filter(
-      (r) => r.success
-    );
+
     if (failedTransactions.length > 0) {
       //Uh oh. At least one was no good.
       failedTransactions.map((ft) =>
@@ -86,11 +83,10 @@ export function PaymentExportButton({
           }),
         })
       );
-    }
-    if (successfulTransactions.length > 0) {
+    } else {
       const paymentUpdateResponse = await updatePayment({
         variables: {
-          paymentIdList: successfulTransactions.map((st) => st.id),
+          paymentIdList: [paymentId],
           payment: {
             exportedat: new Date(),
           },
@@ -98,11 +94,11 @@ export function PaymentExportButton({
       });
       if (!!!paymentUpdateResponse.errors) {
         notification["success"]({
-          message: t("jobs.successes.exported"),
+          message: t("payments.successes.exported"),
         });
       } else {
         notification["error"]({
-          message: t("jobs.errors.exporting", {
+          message: t("payments.errors.exporting", {
             error: JSON.stringify(paymentUpdateResponse.error),
           }),
         });
@@ -118,7 +114,8 @@ export function PaymentExportButton({
       onClick={handleQbxml}
       loading={loading}
       disabled={disabled}
-      type='dashed'>
+      type="dashed"
+    >
       {t("jobs.actions.export")}
     </Button>
   );
diff --git a/client/src/components/payment-form/payment-form.component.jsx b/client/src/components/payment-form/payment-form.component.jsx
index dea21e6fb..239a5601c 100644
--- a/client/src/components/payment-form/payment-form.component.jsx
+++ b/client/src/components/payment-form/payment-form.component.jsx
@@ -108,10 +108,14 @@ export function PaymentFormComponent({
       >
         <Select disabled={disabled}>
           <Select.Option value="Visa">Visa</Select.Option>
-          <Select.Option value="Mastercard">Mastercard</Select.Option>
-          <Select.Option value="AMEX">AMEX</Select.Option>
+          <Select.Option value="Master Card">Mastercard</Select.Option>
+          <Select.Option value="American Express">
+            American Express
+          </Select.Option>
           <Select.Option value="Discover">Discover</Select.Option>
           <Select.Option value="Cash">Cash</Select.Option>
+          <Select.Option value="Cheque">Cheque</Select.Option>
+          <Select.Option value="Interac Debit">Interac Debit</Select.Option>
           <Select.Option value="EFT">EFT</Select.Option>
         </Select>
       </Form.Item>
diff --git a/client/src/components/payments-export-all-button/payments-export-all-button.component.jsx b/client/src/components/payments-export-all-button/payments-export-all-button.component.jsx
index 57b8ef356..daed59cc2 100644
--- a/client/src/components/payments-export-all-button/payments-export-all-button.component.jsx
+++ b/client/src/components/payments-export-all-button/payments-export-all-button.component.jsx
@@ -1,6 +1,7 @@
 import { useMutation } from "@apollo/react-hooks";
 import { Button, notification } from "antd";
 import axios from "axios";
+import _ from "lodash";
 import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { connect } from "react-redux";
@@ -67,44 +68,47 @@ export function PaymentsExportAllButton({
       return;
     }
 
-    console.log("handleQbxml -> PartnerResponse", PartnerResponse);
-    const failedTransactions = PartnerResponse.data.filter((r) => !r.success);
-    const successfulTransactions = PartnerResponse.data.filter(
-      (r) => r.success
-    );
-    if (failedTransactions.length > 0) {
-      //Uh oh. At least one was no good.
-      failedTransactions.map((ft) =>
-        notification["error"]({
-          message: t("payments.errors.exporting", {
-            error: ft.errorMessage || "",
-          }),
-        })
-      );
-    }
-    if (successfulTransactions.length > 0) {
-      const paymentUpdateResponse = await updatePayments({
-        variables: {
-          paymentIdList: successfulTransactions.map((st) => st.id),
-          payment: {
-            //exported: true,
-            exportedat: new Date(),
-          },
-        },
-      });
-      if (!!!paymentUpdateResponse.errors) {
-        notification["success"]({
-          message: t("jobs.successes.exported"),
-        });
-      } else {
-        notification["error"]({
-          message: t("jobs.errors.exporting", {
-            error: JSON.stringify(paymentUpdateResponse.error),
-          }),
-        });
-      }
-    }
+    const groupedData = _.groupBy(PartnerResponse.data, "id");
+    const proms = [];
+    Object.keys(groupedData).forEach((key) => {
+      proms.push(
+        (async () => {
+          const failedTransactions = groupedData[key].filter((r) => !r.success);
 
+          if (failedTransactions.length > 0) {
+            //Uh oh. At least one was no good.
+            failedTransactions.map((ft) =>
+              notification["error"]({
+                message: t("payments.errors.exporting", {
+                  error: ft.errorMessage || "",
+                }),
+              })
+            );
+          } else {
+            const paymentUpdateResponse = await updatePayments({
+              variables: {
+                paymentIdList: [key],
+                payment: {
+                  exportedat: new Date(),
+                },
+              },
+            });
+            if (!!!paymentUpdateResponse.errors) {
+              notification["success"]({
+                message: t("payments.successes.exported"),
+              });
+            } else {
+              notification["error"]({
+                message: t("payments.errors.exporting", {
+                  error: JSON.stringify(paymentUpdateResponse.error),
+                }),
+              });
+            }
+          }
+        })()
+      );
+    });
+    await Promise.all(proms);
     if (!!completedCallback) completedCallback([]);
     if (!!loadingCallback) loadingCallback(false);
     setLoading(false);
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index e86c5175e..fec907fab 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1439,6 +1439,7 @@
 				"totalpayments": "Total Payments"
 			},
 			"successes": {
+				"exported": "Payment exported successfully.",
 				"payment": "Payment created successfully. ",
 				"stripe": "Credit card transaction charged successfully."
 			}
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index 278cca9c5..92859301a 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -1439,6 +1439,7 @@
 				"totalpayments": ""
 			},
 			"successes": {
+				"exported": "",
 				"payment": "",
 				"stripe": ""
 			}
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index ccdc1a100..445626743 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -1439,6 +1439,7 @@
 				"totalpayments": ""
 			},
 			"successes": {
+				"exported": "",
 				"payment": "",
 				"stripe": ""
 			}
diff --git a/server/accounting/qbxml/qbxml-payables.js b/server/accounting/qbxml/qbxml-payables.js
index 3996d5ec7..0e392eed2 100644
--- a/server/accounting/qbxml/qbxml-payables.js
+++ b/server/accounting/qbxml/qbxml-payables.js
@@ -61,7 +61,8 @@ const generateBill = (bill) => {
               FullName: bill.vendor.name,
             },
             TxnDate: moment(bill.date).format("YYYY-MM-DD"),
-            DueDate: moment(bill.due_date).format("YYYY-MM-DD"),
+            DueDate:
+              bill.due_date && moment(bill.due_date).format("YYYY-MM-DD"),
             RefNumber: bill.bill_number,
             Memo: `RO ${bill.job.ro_number || ""} OWNER ${
               bill.job.ownr_fn || ""
diff --git a/server/accounting/qbxml/qbxml-payments.js b/server/accounting/qbxml/qbxml-payments.js
index 44f2178df..e6a4453ef 100644
--- a/server/accounting/qbxml/qbxml-payments.js
+++ b/server/accounting/qbxml/qbxml-payments.js
@@ -6,6 +6,7 @@ const Dinero = require("dinero.js");
 var builder = require("xmlbuilder");
 const moment = require("moment");
 const QbXmlUtils = require("./qbxml-utils");
+const QbxmlReceivables = require("./qbxml-receivables");
 require("dotenv").config({
   path: path.resolve(
     process.cwd(),
@@ -31,42 +32,49 @@ exports.default = async (req, res) => {
       .request(queries.QUERY_PAYMENTS_FOR_EXPORT, {
         payments: paymentsToQuery,
       });
-    const { payments } = result;
+    const { payments, bodyshops } = result;
+    const bodyshop = bodyshops[0];
+    const isThreeTier = bodyshop.accountingconfig.tiers === 3;
+    const twoTierPref = bodyshop.accountingconfig.twotierpref;
 
-    if (isThreeTier) {
-      QbXmlToExecute.push({
-        id: jobs_by_pk.id,
-        okStatusCodes: ["0", "3100"],
-        qbxml: generateSourceCustomerQbxml(jobs_by_pk, bodyshop), // Create the source customer.
-      });
-    }
-
-    QbXmlToExecute.push({
-      id: jobs_by_pk.id,
-      okStatusCodes: ["0", "3100"],
-      qbxml: generateJobQbxml(
-        jobs_by_pk,
-        bodyshop,
-        isThreeTier,
-        2,
-        twoTierPref
-      ),
-    });
-
-    QbXmlToExecute.push({
-      id: jobs_by_pk.id,
-      okStatusCodes: ["0", "3100"],
-      qbxml: generateJobQbxml(
-        jobs_by_pk,
-        bodyshop,
-        isThreeTier,
-        3,
-        twoTierPref
-      ),
-    });
 
     const QbXmlToExecute = [];
     payments.map((i) => {
+
+      if (isThreeTier) {
+        QbXmlToExecute.push({
+          id: i.id,
+          okStatusCodes: ["0", "3100"],
+          qbxml: QbxmlReceivables.generateSourceCustomerQbxml(i.job, bodyshop), // Create the source customer.
+        });
+      }
+  
+      QbXmlToExecute.push({
+        id: i.id,
+        okStatusCodes: ["0", "3100"],
+        qbxml: QbxmlReceivables.generateJobQbxml(
+          i.job,
+          bodyshop,
+          isThreeTier,
+          2,
+          twoTierPref
+        ),
+      });
+  
+      QbXmlToExecute.push({
+        id: i.id,
+        okStatusCodes: ["0", "3100"],
+        qbxml: QbxmlReceivables.generateJobQbxml(
+          i.job,
+          bodyshop,
+          isThreeTier,
+          3,
+          twoTierPref
+        ),
+      });
+
+
+
       QbXmlToExecute.push({
         id: i.id,
         okStatusCodes: ["0"],
diff --git a/server/accounting/qbxml/qbxml-receivables.js b/server/accounting/qbxml/qbxml-receivables.js
index 7db916b6c..4482e2695 100644
--- a/server/accounting/qbxml/qbxml-receivables.js
+++ b/server/accounting/qbxml/qbxml-receivables.js
@@ -29,8 +29,7 @@ exports.default = async (req, res) => {
     const result = await client
       .setHeaders({ Authorization: BearerToken })
       .request(queries.QUERY_JOBS_FOR_RECEIVABLES_EXPORT, { ids: jobIds });
-    const { jobs } = result;
-    const { bodyshops } = result;
+    const { jobs, bodyshops } = result;
     const QbXmlToExecute = [];
 
     const bodyshop = bodyshops[0];
@@ -119,7 +118,7 @@ const generateSourceCustomerQbxml = (jobs_by_pk, bodyshop) => {
 
   return customerQbxml_Full;
 };
-
+exports.generateSourceCustomerQbxml = generateSourceCustomerQbxml;
 const generateJobQbxml = (
   jobs_by_pk,
   bodyshop,
@@ -170,7 +169,7 @@ const generateJobQbxml = (
   console.log("jobQbxml_Full", jobQbxml_Full);
   return jobQbxml_Full;
 };
-
+exports.generateJobQbxml = generateJobQbxml;
 const generateInvoiceQbxml = (jobs_by_pk, bodyshop) => {
   //Build the Invoice XML file.
   const InvoiceLineAdd = [];
diff --git a/server/graphql-client/queries.js b/server/graphql-client/queries.js
index 175d5fb0f..03777d597 100644
--- a/server/graphql-client/queries.js
+++ b/server/graphql-client/queries.js
@@ -146,7 +146,12 @@ query QUERY_BILLS_FOR_PAYABLES_EXPORT($bills: [uuid!]!) {
   `;
 
 exports.QUERY_PAYMENTS_FOR_EXPORT = `
-  query QUERY_PAYMENTS_FOR_EXPORT($payments: [uuid!]!) {
+  query QUERY_PAYMENTS_FOR_EXPORT($payments: [uuid!]!) 
+  {  bodyshops(where: {associations: {active: {_eq: true}}}) {
+    id
+    md_responsibility_centers
+    accountingconfig
+  }
     payments(where: {id: {_in: $payments}}) {
       id
       created_at