magento-engcom/import-export-improvements#49: Add failure if sku is empty value constant

Author: dmanners

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

@@ -444,7 +444,11 @@ public function isRowValid(array $rowData, $rowNum, $isNewProduct = true)
         $error = false;
         $rowScope = $this->_entityModel->getRowScope($rowData);
         if (\Magento\CatalogImportExport\Model\Import\Product::SCOPE_NULL != $rowScope
-            && !empty($rowData[\Magento\CatalogImportExport\Model\Import\Product::COL_SKU])
+            && (
+                !empty($rowData[\Magento\CatalogImportExport\Model\Import\Product::COL_SKU])
+                || $rowData[\Magento\CatalogImportExport\Model\Import\Product::COL_SKU]
+                    !== $this->_entityModel->getEmptyAttributeValueConstant()
+            )
         ) {
             foreach ($this->_getProductAttributes($rowData) as $attrCode => $attrParams) {
                 // check value for non-empty in the case of required attribute?

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

@@ -150,7 +150,12 @@ public function isRequiredAttributeValid($attrCode, array $attributeParams, arra
             $doCheck = true;
         }
 
-        return $doCheck ? isset($rowData[$attrCode]) && strlen(trim($rowData[$attrCode])) : true;
+        if ($doCheck === true) {
+            return isset($rowData[$attrCode])
+                && strlen(trim($rowData[$attrCode]))
+                && trim($rowData[$attrCode]) !== $this->context->getEmptyAttributeValueConstant();
+        }
+        return true;
     }
 
     /**