From d52426f5f5f5411a2d270b75dd1af42ffadacd1c Mon Sep 17 00:00:00 2001
From: Patrick Fic <patrick@imexsystems.ca>
Date: Wed, 28 May 2025 15:21:42 -0700
Subject: [PATCH] IO-3239 Additional logging fixes.

---
 server/accounting/qbo/qbo-receivables.js | 11 +++++++----
 server/utils/logger.js                   |  2 +-
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/server/accounting/qbo/qbo-receivables.js b/server/accounting/qbo/qbo-receivables.js
index e2a0fa307..008e4b3f0 100644
--- a/server/accounting/qbo/qbo-receivables.js
+++ b/server/accounting/qbo/qbo-receivables.js
@@ -478,7 +478,7 @@ async function InsertJob(oauthClient, qbo_realmId, req, job, parentTierRef) {
 
 exports.InsertJob = InsertJob;
 
-async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid) {
+async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid, jobid) {
   const items = await oauthClient.makeApiCall({
     url: urlBuilder(qbo_realmId, "query", `select * From Item where active=true maxresults 1000`),
     method: "POST",
@@ -492,6 +492,7 @@ async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid) {
     name: "QueryItems",
     status: items.response?.status,
     bodyshopid,
+    jobid: jobid,
     email: req.user.email
   })
   setNewRefreshToken(req.user.email, items);
@@ -508,6 +509,7 @@ async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid) {
     name: "QueryTaxCodes",
     status: taxCodes.response?.status,
     bodyshopid,
+    jobid: jobid,
     email: req.user.email
   })
   const classes = await oauthClient.makeApiCall({
@@ -523,6 +525,7 @@ async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid) {
     name: "QueryClasses",
     status: classes.response?.status,
     bodyshopid,
+    jobid: jobid,
     email: req.user.email
   })
   const taxCodeMapping = {};
@@ -559,7 +562,7 @@ async function QueryMetaData(oauthClient, qbo_realmId, req, bodyshopid) {
 }
 
 async function InsertInvoice(oauthClient, qbo_realmId, req, job, bodyshop, parentTierRef) {
-  const { items, taxCodes, classes } = await QueryMetaData(oauthClient, qbo_realmId, req, job.shopid);
+  const { items, taxCodes, classes } = await QueryMetaData(oauthClient, qbo_realmId, req, job.shopid, job.id);
   const InvoiceLineAdd = CreateInvoiceLines({
     bodyshop,
     jobs_by_pk: job,
@@ -653,7 +656,7 @@ async function InsertInvoice(oauthClient, qbo_realmId, req, job, bodyshop, paren
       platform: "QBO",
       method: "POST",
       name: "InsertInvoice",
-      status: result.status,
+      status: result.response?.status,
       bodyshopid: job.shopid,
       jobid: job.id,
       email: req.user.email
@@ -778,7 +781,7 @@ async function InsertInvoiceMultiPayerInvoice(
       platform: "QBO",
       method: "POST",
       name: "InsertInvoice",
-      status: result.response.status,
+      status: result.response?.status,
       bodyshopid: job.shopid,
       jobid: job.id,
       email: req.user.email
diff --git a/server/utils/logger.js b/server/utils/logger.js
index 41d71cea6..ccb2e905e 100644
--- a/server/utils/logger.js
+++ b/server/utils/logger.js
@@ -206,7 +206,7 @@ const createLogger = () => {
             jobid,
             paymentid,
             billid,
-            status: status.toString() ?? "0",
+            status: status?.toString() ?? "0",
             bodyshopid,
             email
           }