From b9b6759c544bdf68544cf9bff5c5b0035faf4d8e Mon Sep 17 00:00:00 2001
From: Patrick Fic <>
Date: Thu, 30 Jun 2022 09:04:17 -0700
Subject: [PATCH] IO-1967 Update audit trail for labor adjustments.

---
 bodyshop_translations.babel                   |  2 +-
 .../bill-enter-modal.container.jsx            | 12 ++++++---
 .../job-line-convert-to-labor.component.jsx   | 12 +++++++--
 ...-allocations-adjustment-edit.component.jsx | 26 ++++++++++++++++++-
 .../parts-order-modal.container.jsx           | 11 ++++----
 client/src/translations/en_us/common.json     |  2 +-
 client/src/utils/AuditTrailMappings.js        |  3 ++-
 7 files changed, 52 insertions(+), 16 deletions(-)

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index 0a0bddd67..df2b5f8be 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
diff --git a/client/src/components/bill-enter-modal/bill-enter-modal.container.jsx b/client/src/components/bill-enter-modal/bill-enter-modal.container.jsx
index 82948fe88..e21cb8efd 100644
--- a/client/src/components/bill-enter-modal/bill-enter-modal.container.jsx
+++ b/client/src/components/bill-enter-modal/bill-enter-modal.container.jsx
@@ -144,6 +144,14 @@ function BillEnterModalContainer({
       adjKeys.forEach((key) => {
         newAdjustments[key] =
           (newAdjustments[key] || 0) + adjustmentsToInsert[key];
+
+        insertAuditTrail({
+          jobid: values.jobid,
+          operation: AuditTrailMapping.jobmodifylbradj({
+            mod_lbr_ty: key,
+            hours: adjustmentsToInsert[key],
+          }),
+        });
       });
 
       const jobUpdate = client.mutate({
@@ -161,10 +169,6 @@ function BillEnterModalContainer({
         });
         return;
       }
-      insertAuditTrail({
-        jobid: values.jobid,
-        operation: AuditTrailMapping.jobmodifylbradj(),
-      });
     }
 
     const markPolReceived =
diff --git a/client/src/components/job-line-convert-to-labor/job-line-convert-to-labor.component.jsx b/client/src/components/job-line-convert-to-labor/job-line-convert-to-labor.component.jsx
index 573518ce3..f7ff2f543 100644
--- a/client/src/components/job-line-convert-to-labor/job-line-convert-to-labor.component.jsx
+++ b/client/src/components/job-line-convert-to-labor/job-line-convert-to-labor.component.jsx
@@ -36,7 +36,12 @@ export default connect(
   mapDispatchToProps
 )(JobLineConvertToLabor);
 
-export function JobLineConvertToLabor({ jobline, job, ...otherBtnProps }) {
+export function JobLineConvertToLabor({
+  jobline,
+  job,
+  insertAuditTrail,
+  ...otherBtnProps
+}) {
   const { t } = useTranslation();
 
   const [loading, setLoading] = useState(false);
@@ -99,7 +104,10 @@ export function JobLineConvertToLabor({ jobline, job, ...otherBtnProps }) {
     }
     insertAuditTrail({
       jobid: values.jobid,
-      operation: AuditTrailMapping.jobmodifylbradj(),
+      operation: AuditTrailMapping.jobmodifylbradj({
+        hours: calculateAdjustment({ mod_lbr_ty, job, jobline }),
+        mod_lbr_ty,
+      }),
     });
     setLoading(false);
     setVisibility(false);
diff --git a/client/src/components/labor-allocations-adjustment-edit/labor-allocations-adjustment-edit.component.jsx b/client/src/components/labor-allocations-adjustment-edit/labor-allocations-adjustment-edit.component.jsx
index c26f78983..6f9aeb680 100644
--- a/client/src/components/labor-allocations-adjustment-edit/labor-allocations-adjustment-edit.component.jsx
+++ b/client/src/components/labor-allocations-adjustment-edit/labor-allocations-adjustment-edit.component.jsx
@@ -12,7 +12,24 @@ import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { UPDATE_JOB } from "../../graphql/jobs.queries";
 
-export default function LaborAllocationsAdjustmentEdit({
+import { connect } from "react-redux";
+import { createStructuredSelector } from "reselect";
+import { insertAuditTrail } from "../../redux/application/application.actions";
+import AuditTrailMapping from "../../utils/AuditTrailMappings";
+const mapStateToProps = createStructuredSelector({
+  //currentUser: selectCurrentUser
+});
+const mapDispatchToProps = (dispatch) => ({
+  insertAuditTrail: ({ jobid, operation }) =>
+    dispatch(insertAuditTrail({ jobid, operation })),
+});
+export default connect(
+  mapStateToProps,
+  mapDispatchToProps
+)(LaborAllocationsAdjustmentEdit);
+
+export function LaborAllocationsAdjustmentEdit({
+  insertAuditTrail,
   jobId,
   mod_lbr_ty,
   adjustments,
@@ -51,6 +68,13 @@ export default function LaborAllocationsAdjustmentEdit({
       notification["success"]({
         message: t("jobs.successes.save"),
       });
+      insertAuditTrail({
+        jobid: values.jobid,
+        operation: AuditTrailMapping.jobmodifylbradj({
+          mod_lbr_ty: values.mod_lbr_ty,
+          hours: values.hours - ((adjustments && adjustments[mod_lbr_ty]) || 0),
+        }),
+      });
     }
     setLoading(false);
     setVisible(false);
diff --git a/client/src/components/parts-order-modal/parts-order-modal.container.jsx b/client/src/components/parts-order-modal/parts-order-modal.container.jsx
index c9fa7bc43..ec7cc9470 100644
--- a/client/src/components/parts-order-modal/parts-order-modal.container.jsx
+++ b/client/src/components/parts-order-modal/parts-order-modal.container.jsx
@@ -125,6 +125,11 @@ export function PartsOrderModalContainer({
         });
         return;
       }
+      notification["success"]({
+        message: values.isReturn
+          ? t("parts_orders.successes.return_created")
+          : t("parts_orders.successes.created"),
+      });
       insertAuditTrail({
         jobid: jobId,
         operation: isReturn
@@ -168,12 +173,6 @@ export function PartsOrderModalContainer({
       });
     }
 
-    notification["success"]({
-      message: values.isReturn
-        ? t("parts_orders.successes.return_created")
-        : t("parts_orders.successes.created"),
-    });
-
     if (values.vendorid === bodyshop.inhousevendorid) {
       setBillEnterContext({
         actions: { refetch: refetch },
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index 7562a7ddd..12cf52836 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -103,7 +103,7 @@
 				"jobimported": "Job imported.",
 				"jobinproductionchange": "Job production status set to {{inproduction}}",
 				"jobioucreated": "IOU Created.",
-				"jobmodifylbradj": "Labor adjustments modified.",
+				"jobmodifylbradj": "Labor adjustments modified {{mod_lbr_ty}} / {{hours}}.",
 				"jobnoteadded": "Note added to job.",
 				"jobnotedeleted": "Note deleted from job.",
 				"jobnoteupdated": "Note updated on job.",
diff --git a/client/src/utils/AuditTrailMappings.js b/client/src/utils/AuditTrailMappings.js
index 031cab1a0..e27254d36 100644
--- a/client/src/utils/AuditTrailMappings.js
+++ b/client/src/utils/AuditTrailMappings.js
@@ -18,7 +18,8 @@ const AuditTrailMapping = {
     i18n.t("audit_trail.messages.jobspartsorder", { order_number }),
   jobspartsreturn: (order_number) =>
     i18n.t("audit_trail.messages.jobspartsreturn", { order_number }),
-  jobmodifylbradj: () => i18n.t("audit_trail.messages.jobmodifylbradj", {}),
+  jobmodifylbradj: ({ mod_lbr_ty, hours }) =>
+    i18n.t("audit_trail.messages.jobmodifylbradj", { mod_lbr_ty, hours }),
   billposted: (invoice_number) =>
     i18n.t("audit_trail.messages.billposted", { invoice_number }),
   billupdated: (invoice_number) =>