From c95448f5cc80adc3f9be326d7bec4b63d28e2a66 Mon Sep 17 00:00:00 2001 From: Yonn Trimoreau Date: Tue, 27 Sep 2016 16:47:56 +0200 Subject: [PATCH] Fixed "additional_images" column validation When importing (for real), the character used to explode "additional_images" column is not the same as in validation. Instead, the multiple value separator is used. Here you can find the corresponding line for real import : https://github.com/pepe1518/magento2/blob/2ee22c350530e97caf33a079bcad86cfe8b7da40/vendor/magento/module-catalog-import-export/Model/Import/Product.php#L1330 --- .../Model/Import/Product/Validator/Media.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/vendor/magento/module-catalog-import-export/Model/Import/Product/Validator/Media.php b/vendor/magento/module-catalog-import-export/Model/Import/Product/Validator/Media.php index c5c96ea23..f28afaab7 100755 --- a/vendor/magento/module-catalog-import-export/Model/Import/Product/Validator/Media.php +++ b/vendor/magento/module-catalog-import-export/Model/Import/Product/Validator/Media.php @@ -16,8 +16,6 @@ class Media extends AbstractImportValidator implements RowValidatorInterface const ADDITIONAL_IMAGES = 'additional_images'; - const ADDITIONAL_IMAGES_DELIMITER = ','; - /** @var array */ protected $mediaAttributes = ['image', 'small_image', 'thumbnail']; @@ -83,7 +81,7 @@ public function isValid($value) } } if (isset($value[self::ADDITIONAL_IMAGES]) && strlen($value[self::ADDITIONAL_IMAGES])) { - foreach (explode(self::ADDITIONAL_IMAGES_DELIMITER, $value[self::ADDITIONAL_IMAGES]) as $image) { + foreach (explode($this->context->getMultipleValueSeparator(), $value[self::ADDITIONAL_IMAGES]) as $image) { if (!$this->checkPath($image) && !$this->checkValidUrl($image)) { $this->_addMessages( [