From df423192bfc5e2389872147e92b7d22e648a2927 Mon Sep 17 00:00:00 2001
From: Blake Matthes <cowchmonkey@gmail.com>
Date: Wed, 27 Aug 2025 17:17:03 -0500
Subject: [PATCH] refactor(labeling): moved bookin fails inside bookin as it
 could be off

---
 .../ocp/controller/labeling/labelProcess.ts   | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/server/services/ocp/controller/labeling/labelProcess.ts b/server/services/ocp/controller/labeling/labelProcess.ts
index f2593ca..a4a2e4a 100644
--- a/server/services/ocp/controller/labeling/labelProcess.ts
+++ b/server/services/ocp/controller/labeling/labelProcess.ts
@@ -277,26 +277,26 @@ export const labelingProcess = async ({
     let book: any = [];
     if (bookin[0].value === "1") {
         book = await bookInLabel(label.data);
+
+        if (!book.success) {
+            // createLog(
+            //     "error",
+            //     "labeling",
+            //     "ocp",
+            //     `Error Booking in label: ${book.errors[0].message}`
+            // );
+            return {
+                success: false,
+                message: `Error Booking in label: ${book.errors[0]?.message}`,
+                data: book,
+            };
+        }
     } else {
         createLog("info", "labeling", "ocp", "Bookin is turned off.");
 
         // will add later!!! :P
     }
 
-    if (!book.success) {
-        // createLog(
-        //     "error",
-        //     "labeling",
-        //     "ocp",
-        //     `Error Booking in label: ${book.errors[0].message}`
-        // );
-        return {
-            success: false,
-            message: `Error Booking in label: ${book.errors[0].message}`,
-            data: book,
-        };
-    }
-
     // inhouse - if the inhouse funtion is turned on we will deliver to inhouse as long as we did not hit an error state
     const inhouseDelivery = settingData.filter(
         (s) => s.name === "inhouseDelivery"