MAGETWO-67240: Duplicate and broken products appear after import

Author: andimov

app/code/Magento/CatalogImportExport/Model/Import/Product.php

@@ -2480,9 +2480,9 @@ public function validateRow(array $rowData, $rowNum)
                 $productUrlSuffix = $this->getProductUrlSuffix($storeId);
                 $urlPath = $urlKey . $productUrlSuffix;
                 if (empty($this->urlKeys[$storeId][$urlPath])
-                    || ($this->urlKeys[$storeId][$urlPath] == $sku)
+                    || ($this->urlKeys[$storeId][$urlPath] == strtolower($sku))
                 ) {
-                    $this->urlKeys[$storeId][$urlPath] = $sku;
+                    $this->urlKeys[$storeId][$urlPath] = strtolower($sku);
                     $this->rowNumbers[$storeId][$urlPath] = $rowNum;
                 } else {
                     $message = sprintf(

dev/tests/integration/testsuite/Magento/CatalogImportExport/Model/Import/ProductTest.php

@@ -1810,9 +1810,9 @@ public function testImportWithDifferentSkuCase()
             'simple3' => 58,
         ];
         $updatedPrices = [
-            'simple1' => 999,
-            'simple2' => 999,
-            'simple3' => 999,
+            'simple1' => 111,
+            'simple2' => 222,
+            'simple3' => 333,
         ];
 
         $filesystem = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
@@ -1838,6 +1838,10 @@ public function testImportWithDifferentSkuCase()
 
         $this->_model->importData();
 
+        $this->assertEquals(
+            3,
+            count($productRepository->getList($searchCriteria)->getItems())
+        );
         foreach ($importedPrices as $sku => $expectedPrice) {
             $this->assertEquals($expectedPrice, $productRepository->get($sku)->getPrice());
         }
@@ -1860,6 +1864,11 @@ public function testImportWithDifferentSkuCase()
 
         $this->_model->importData();
 
+        $this->assertEquals(
+            3,
+            count($productRepository->getList($searchCriteria)->getItems()),
+            'Ensures that new products were not created'
+        );
         foreach ($updatedPrices as $sku => $expectedPrice) {
             $this->assertEquals(
                 $expectedPrice,

dev/tests/integration/testsuite/Magento/CatalogImportExport/Model/Import/_files/products_to_import_with_changed_sku_case.csv

@@ -1,4 +1,6 @@
 sku,product_type,store_view_code,name,price,attribute_set_code,categories
-simPLE1,simple,,"simple 1",999,Default,"Default Category/Category 1"
-SIMPLE2,simple,,"simple 2",999,Default,"Default Category/Category 1"
-Simple3,simple,,"simple 3",999,Default,"Default Category/Category 1"
+simPLE1,simple,,"simple 1",111,Default,"Default Category/Category 1"
+SIMPLE2,simple,,"simple 2",111,Default,"Default Category/Category 1"
+Simple3,simple,,"simple 3",111,Default,"Default Category/Category 1"
+SiMPle3,simple,,"simple 3",333,Default,"Default Category/Category 1"
+SIMplE2,simple,,"simple 2",222,Default,"Default Category/Category 1"