Skip to content

Commit d6f672a

Browse files
author
Sergey Shvets
committed
MAGETWO-97345: [Magento Cloud] Import doesn't work with skip error entries
1 parent a305a4f commit d6f672a

File tree

2 files changed

+2
-9
lines changed

2 files changed

+2
-9
lines changed

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

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1649,11 +1649,8 @@ protected function _saveProducts()
16491649
continue;
16501650
}
16511651
if ($this->getErrorAggregator()->hasToBeTerminated()) {
1652-
$validationStrategy = $this->_parameters[Import::FIELD_NAME_VALIDATION_STRATEGY];
1653-
if (ProcessingErrorAggregatorInterface::VALIDATION_STRATEGY_SKIP_ERRORS !== $validationStrategy) {
1654-
$this->getErrorAggregator()->addRowToSkip($rowNum);
1655-
continue;
1656-
}
1652+
$this->getErrorAggregator()->addRowToSkip($rowNum);
1653+
continue;
16571654
}
16581655
$rowScope = $this->getRowScope($rowData);
16591656

app/code/Magento/ImportExport/Model/Import.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -621,10 +621,6 @@ public function validateSource(\Magento\ImportExport\Model\Import\AbstractSource
621621
$this->addLogComment($messages);
622622

623623
$result = !$errorAggregator->getErrorsCount();
624-
$validationStrategy = $this->getData(self::FIELD_NAME_VALIDATION_STRATEGY);
625-
if ($validationStrategy === ProcessingErrorAggregatorInterface::VALIDATION_STRATEGY_SKIP_ERRORS) {
626-
$result = true;
627-
}
628624

629625
if ($result) {
630626
$this->addLogComment(__('Import data validation is complete.'));

0 commit comments

Comments
 (0)