Skip to content

Commit 4a49e29

Browse files
committed
Added _media_attribute_id validation during the first step of the import procoess
1 parent 5b266f6 commit 4a49e29

File tree

1 file changed

+21
-1
lines changed
  • app/code/core/Mage/ImportExport/Model/Import/Entity

1 file changed

+21
-1
lines changed

app/code/core/Mage/ImportExport/Model/Import/Entity/Product.php

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@ class Mage_ImportExport_Model_Import_Entity_Product extends Mage_ImportExport_Mo
9494
const ERROR_GROUP_PRICE_DATA_INCOMPLETE = 'groupPriceDataIsIncomplete';
9595
const ERROR_SKU_NOT_FOUND_FOR_DELETE = 'skuNotFoundToDelete';
9696
const ERROR_SUPER_PRODUCTS_SKU_NOT_FOUND = 'superProductsSkuNotFound';
97+
const ERROR_MEDIA_DATA_INCOMPLETE = 'mediaDataIsIncomplete';
9798

9899
/**
99100
* Pairs of attribute set ID-to-name.
@@ -182,7 +183,8 @@ class Mage_ImportExport_Model_Import_Entity_Product extends Mage_ImportExport_Mo
182183
self::ERROR_INVALID_TIER_PRICE_GROUP => 'Tier Price customer group ID is invalid',
183184
self::ERROR_TIER_DATA_INCOMPLETE => 'Tier Price data is incomplete',
184185
self::ERROR_SKU_NOT_FOUND_FOR_DELETE => 'Product with specified SKU not found',
185-
self::ERROR_SUPER_PRODUCTS_SKU_NOT_FOUND => 'Product with specified super products SKU not found'
186+
self::ERROR_SUPER_PRODUCTS_SKU_NOT_FOUND => 'Product with specified super products SKU not found',
187+
self::ERROR_MEDIA_DATA_INCOMPLETE => 'Media data is incomplete'
186188
);
187189

188190
/**
@@ -693,6 +695,23 @@ protected function _isSuperProductsSkuValid($rowData, $rowNum)
693695
return true;
694696
}
695697

698+
/**
699+
* Check media information
700+
*
701+
* @param array $rowData
702+
* @param int $rowNum
703+
* @return bool
704+
*/
705+
protected function _isMediaValid($rowData, $rowNum)
706+
{
707+
if (!empty($rowData['_media_image']) && empty($rowData['_media_attribute_id'])) {
708+
$this->addRowError(self::ERROR_MEDIA_DATA_INCOMPLETE, $rowNum);
709+
return false;
710+
}
711+
712+
return true;
713+
}
714+
696715
/**
697716
* Gather and save information about product links.
698717
* Must be called after ALL products saving done.
@@ -1567,6 +1586,7 @@ public function validateRow(array $rowData, $rowNum)
15671586
$this->_isTierPriceValid($rowData, $rowNum);
15681587
$this->_isGroupPriceValid($rowData, $rowNum);
15691588
$this->_isSuperProductsSkuValid($rowData, $rowNum);
1589+
$this->_isMediaValid($rowData, $rowNum);
15701590

15711591
if (self::SCOPE_DEFAULT == $rowScope) { // SKU is specified, row is SCOPE_DEFAULT, new product block begins
15721592
$this->_processedEntitiesCount ++;

0 commit comments

Comments
 (0)