Bug fix: Remove fallback from LCSC barcode part resolver (#1302)

Author: swdee

src/Services/LabelSystem/BarcodeScanner/BarcodeScanResultHandler.php

@@ -217,8 +217,8 @@ private function resolvePartFromVendor(EIGP114BarcodeScanResult $barcodeScan) :
      * Resolve LCSC barcode -> Part.
      * Strategy:
      *  1) Try providerReference.provider_id == pc (LCSC "Cxxxxxx") if you store it there
-     *  2) Fallback to manufacturer_product_number == pm (MPN)
      * Returns first match (consistent with EIGP114 logic)
+     * 2) Fallback to search across supplier part number (SPN)
      */
     private function resolvePartFromLCSC(LCSCBarcodeScanResult $barcodeScan): ?Part
     {
@@ -231,13 +231,8 @@ private function resolvePartFromLCSC(LCSCBarcodeScanResult $barcodeScan): ?Part
             }
         }
 
-        // Fallback to MPN (pm)
-        $pm = $barcodeScan->mpn; // e.g. RC0402FR-071ML
-        if (!$pm) {
-            return null;
-        }
-
-        return $this->em->getRepository(Part::class)->getPartByMPN($pm);
+        // fallback to search by SPN
+        return $this->em->getRepository(Part::class)->getPartBySPN($pc);
     }
 
 

tests/Services/LabelSystem/BarcodeScanner/BarcodeScanResultHandlerTest.php

@@ -115,8 +115,8 @@ public function testEIGPBarcodeResolvePartOrNullReturnsNullWhenNotFound(): void
     public function testLCSCBarcodeResolvePartOrNullReturnsNullWhenNotFound(): void
     {
         $scan = new LCSCBarcodeScanResult(
-            fields: ['pc' => 'C0000000', 'pm' => ''],
-            rawInput: '{pc:C0000000,pm:}'
+            fields: ['pc' => 'C0000000', 'pm' => 'NON_EXISTENT_MPN_12345'],
+            rawInput: '{pc:C0000000,pm:NON_EXISTENT_MPN_12345}'
         );
 
         $this->assertNull($this->service->resolvePart($scan));