From d389136223d4261f9d08bf8bc0cb2c79ac104d6b Mon Sep 17 00:00:00 2001
From: dhanabalan <dhana@example.com>
Date: Mon, 1 Dec 2025 13:58:15 +0530
Subject: [PATCH] Refactor processSerialNumber method to improve package
 validation logic

---
 .../Pages/CreateInvoiceValidation.php         | 23 +++++++++++--------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/app/Filament/Resources/InvoiceValidationResource/Pages/CreateInvoiceValidation.php b/app/Filament/Resources/InvoiceValidationResource/Pages/CreateInvoiceValidation.php
index 42a71aa78..450195543 100644
--- a/app/Filament/Resources/InvoiceValidationResource/Pages/CreateInvoiceValidation.php
+++ b/app/Filament/Resources/InvoiceValidationResource/Pages/CreateInvoiceValidation.php
@@ -3265,6 +3265,8 @@ class CreateInvoiceValidation extends CreateRecord
                         return;
                     }
 
+                    $invalidPackage = false;
+
                     $hasMotorQr = $record->stickerMasterRelation->tube_sticker_motor ?? null;
                     $hasPumpQr = $record->stickerMasterRelation->tube_sticker_pump ?? null;
                     $hasPumpSetQr = $record->stickerMasterRelation->tube_sticker_pumpset ?? null;
@@ -3274,17 +3276,18 @@ class CreateInvoiceValidation extends CreateRecord
                         $hasMotorQr = $record->stickerMasterRelation->pack_slip_motor ?? null;
                         $hasPumpQr = $record->stickerMasterRelation->pack_slip_pump ?? null;
                         $hasPumpSetQr = $record->stickerMasterRelation->pack_slip_pumpset ?? null;
-                    } elseif (! $hasPumpSetQr && ! $hasPumpQr) {
-                        $hasPumpQr = $record->stickerMasterRelation->pack_slip_pump ?? null;
-                    }
+                    } else {
+                        if (! $hasPumpSetQr && ! $hasPumpQr) {
+                            $hasPumpQr = $record->stickerMasterRelation->pack_slip_pump ?? null;
+                        }
 
-                    $invalidPackage = false;
-                    $hasTubeMotorQr = $record->stickerMasterRelation->tube_sticker_motor ?? null;
-                    $hasPackMotorQr = $record->stickerMasterRelation->pack_slip_motor ?? null;
-                    $hasTubePumpSetQr = $record->stickerMasterRelation->tube_sticker_pumpset ?? null;
-                    $hasPackPumpSetQr = $record->stickerMasterRelation->pack_slip_pumpset ?? null;
-                    if ($hasTubeMotorQr != $hasPackMotorQr || $hasTubePumpSetQr != $hasPackPumpSetQr) {
-                        $invalidPackage = true;
+                        $hasTubeMotorQr = $record->stickerMasterRelation->tube_sticker_motor ?? null;
+                        $hasPackMotorQr = $record->stickerMasterRelation->pack_slip_motor ?? null;
+                        $hasTubePumpSetQr = $record->stickerMasterRelation->tube_sticker_pumpset ?? null;
+                        $hasPackPumpSetQr = $record->stickerMasterRelation->pack_slip_pumpset ?? null;
+                        if ($hasTubeMotorQr != $hasPackMotorQr || $hasTubePumpSetQr != $hasPackPumpSetQr) {
+                            $invalidPackage = true;
+                        }
                     }
 
                     $hadMotorQr = $record->motor_scanned_status ?? null;