Magento – Magento 2 : Error when try to set image for product and save

imagemagento2PHPproduct

When i'm trying to set image for the product

$configurable->setMediaGallery(array('images'=>array (), 'values'=>array ()));
$path = $this->helper->uploadImage($data->main_image);
if ($path && file_exists($path)) {
    $configurable->addImageToMediaGallery($path, array('image', 'small_image', 'thumbnail'), true, false);
}

and then when save it i'm getting an error

Exception #0 (Magento\Framework\Exception\FileSystemException): The file "… pub/media/tmp/catalog/product/8/1/81cz5-izU0L.jpg" doesn't exist or not a file

Looks like some permission issue but i've checked all directories have all needed access.
Any suggestions?
Thanks.

Best Answer

The file path should be relative to media folder. Below I'll try to explain why.

So addImageToMediaGallery calls Model\Product\Gallery\Processor.php::addImage

 $file = $this->mediaDirectory->getRelativePath($file);
 if (!$this->mediaDirectory->isFile($file)) {
      throw new LocalizedException(__('The image does not exist.'));
 }

You can do a debug of the $file, and then keep digging which will eventually bring you to Magento\Framework\Filesystem\Directory\Read

Particularly this line 219: $this->driver->getAbsolutePath($this->path, $path)

Now, if your path was "/fullpath/pub/media/../file.jpg" which you passed to addImageToMediaGallery, then by debugging the result of getAbsolutePath call, you will get something like this:

/fullpath/pub/media/fullpath/pub/media/../file.jpg

So your path is prefixed with the full server path to your media folder and that's make it invalid and therefore exception 'The image does not exist.' is thrown.

Hope it makes it more clear now.

Related Solutions

Magento – Setting image, small_image, and thumbnail for media gallery imports

Try passing an array as the media type. Theer is no need to iterate over each mediaType and setting them individually.

your code would thus be, which will be faster as well.

ini_set('memory_limit', '-1');

require 'app/Mage.php';
Mage::app();

echo "Loading catalog...\n";
$products = Mage::getModel('catalog/product')->setStoreId(13)->getCollection()->addAttributeToSelect('*');
echo count($products) . " products selected for image import.\n";
// Remove unset images, add image to gallery if exists
$importDir = Mage::getBaseDir('media') . DS . 'import/';  

foreach ($products as $product) {
        $sku = Mage::getModel('catalog/product')->load($product->getId())->getSku();
        if (strstr($sku, " ")) {
            echo "Sku $sku contains whitespace, skipping.\n";
            continue;
        }
        $imgFile = $importDir . $sku . ".jpg";
        // Add three image sizes to media gallery
        $mediaArray = array(
            'thumbnail',
            'small_image',
            'image'
        );
        if (file_exists($imgFile)) {
            try {
                $product->addImageToMediaGallery($imgFile, $mediaArray, false, false);
                $product->save();
                echo "Sku: $sku updated.\n";
            } catch (Exception $e) {
                echo $e->getMessage();
            }
        } else {
            echo "Could not match image to $sku. Path was: {$filePath}\n";
            break;
        }
    }

Anyways, I suspect your issue is related to the fact that your product->save was within the media loop of your original code.

For reference, the end-point of the mediaAttribute is this function, located in class Mage_Catalog_Model_Product_Attribute_Backend_Media

/**
     * Set media attribute value
     *
     * @param Mage_Catalog_Model_Product $product
     * @param string|array $mediaAttribute
     * @param string $value
     * @return Mage_Catalog_Model_Product_Attribute_Backend_Media
     */
    public function setMediaAttribute(Mage_Catalog_Model_Product $product, $mediaAttribute, $value)
    {
        $mediaAttributeCodes = array_keys($product->getMediaAttributes());

        if (is_array($mediaAttribute)) {
            foreach ($mediaAttribute as $atttribute) {
                if (in_array($atttribute, $mediaAttributeCodes)) {
                    $product->setData($atttribute, $value);
                }
            }
        } elseif (in_array($mediaAttribute, $mediaAttributeCodes)) {
            $product->setData($mediaAttribute, $value);
        }

        return $this;
    }

Magento – Magento2 Product import (image filenames)

I've also recently encountered this on M2.2.2

After some investigation, I personally believe this is a feature as opposed to a bug as the Magento team appear validating image names as web safe i.e. no spaces or special characters.

Historically when working with M1.x (little/no image name validation) I spent many hours fixing images when clients tried to upload images which were not web safe, so I'm welcoming this validation with open arms.

Best Answer

Related Solutions

Magento – Setting image, small_image, and thumbnail for media gallery imports

Magento – Magento2 Product import (image filenames)

Related Topic