Undefined index "media_type" while adding image to product's media gallery

Question

Here is the code snippet I use to add an image in a product's media gallery :

$sku = 'product-example-sku';
$image_url = 'http://example.com/image.jpg';
$image_ext = pathinfo($image_url, PATHINFO_EXTENSION);
$image_path = '/import/' . $sku . '.' . $image_ext;
file_put_contents('pub/media' . $image_path, file_get_contents($image_url));
if (file_exists('pub/media' . $image_path))
{
    try
    {
        $product->addImageToMediaGallery($image_path, array('image'), true, false);
    }
    catch (\Exception $e)
    {
        echo '[ERROR] Image import failed for product "' . $sku . '": ' . $e->getMessage();
    }
}

$this->_productRepository->save($product);

I am sure that the image is actually downloaded and moved successfully. But when I try to save the product after adding the image, this exception is thrown by Magento:

Exception #0 (Exception): Notice: Undefined index: media_type in /var/www/<website>/vendor/magento/module-catalog/Model/Product.php on line 2527

I can't find in the Magento core's code how to solve my problem.

Any help would be appreciated.

Answer 1

Since addImageToMediaGallery is not added to interface \Magento\Catalog\Api\Data\ProductInterface, I'd not recommend to use addImageToMediaGallery with ProductRepository.

For better practice you can try to use Magento2 Service Layer. You can save a new image using \Magento\Catalog\Api\ProductAttributeMediaGalleryManagementInterface::create

If you will go deep to the code, under the hood of the method the attribute "media_gallery" will be set and then \Magento\Catalog\Api\ProductRepositoryInterface::save will be invoked. And inside of this method the method \Magento\Catalog\Model\Product\Gallery\Processor::addImage will be invoked, which basically invoked, when you run addImageToMediaGallery

Answer 2

Pretty sure you have to initialize the media gallery first. I've done that several times and the code I used was always like this:

->setMediaGallery(
    [
        'images'=>  [],
        'values'=>  []
    ]
) // Media gallery initialization
->addImageToMediaGallery(
    $image,
    [
        'image',
        'thumbnail',
        'small_image'
    ],
    false,
    false) // Assigning image, thumb and small image to media gallery

Answer 3

I found a temporary solution for this bug

You can use product resource model instead of ProductRepository and it works like a charm. Some code sample here:

public function __construct(
    \Magento\Catalog\Model\ProductRepository $productRepository,
    \Magento\Catalog\Model\ResourceModel\Product $productResource
)
{
    $this->productRepository = $productRepository;
    $this->productResource = $productResource;
}
public function addImage() {
    $sku = "product-example-sku";
    $image_path = '/import/' . $sku . '.png';
    $product = $this->productRepository->get($sku);
    $product->addImageToMediaGallery($image_path, array('image'), true, false);
    $this->productResource->save($product);
}

Answer 4

here is the solution to resolve the issue.

vendor\magento\module-catalog\Model\Product\Gallery\Processor.php

Please override the file in your theme.

Please search for

$mediaGalleryData['images'][] = [
            'file' => $fileName,
            'position' => $position,
            'label' => '',
            'disabled' => (int)$exclude,
               ];

and replace with

$mediaGalleryData['images'][] = [
            'file' => $fileName,
            'position' => $position,
            'label' => '',
            'disabled' => (int)$exclude,
            'media_type' => 'image',
            'types'      => $mediaAttribute
        ];

I have just added media_type and types fields here.

I have tested and it's working fine.