Adding video gallery item through API does not set extension attributes

[bartlubbersen]

Preconditions

1. Magento 2.1.2
2. PHP7

Steps to reproduce

1. Go to swagger or call rest API through script
2. Try updating or creating a product with a video_url, video_title etc. with media type "external-video".
3. I used the following JSON:

{
  "product": {
    "sku": "SKU",
    "media_gallery_entries": [
        {
            "media_type": "external-video",
            "disabled": false,
            "label": "",
            "types": [],
            "content": {
                "type": "image/jpeg",
                "name": "FyL6smNEMiU.jpg",
                "base64_encoded_data": "encoded_image"
            },
            "extension_attributes": {
                "video_content": {
                    "media_type": "external-video",
                    "video_url": "https://youtu.be/FyL6smNEMiU",
                    "video_title": "Video title",
                    "video_description": "Video description"
                }
            }
        }
    ]
  },
  "saveOptions": true
}

Ofcourse I used a valid base64 encoded image as thumbnail and used a correct SKU.

Expected result

Product should be created or updated with a valid video gallery item, which should have all necessary attributes filled like video_url and video_title.

Actual result

The product is updated, however there is only a thumbnail image and no video attributes. If you will load the product through API you will get the item back with the right media_type but without attributes.

Comment

I think it has something to do with the fact that the productRepository is handling the gallery entries by itself and is not yet processing any of the extension_attributes for gallery items.

[rafal-kos]

I can confirm the issue. Example data

$data = [
  'store_id' => 0,
  'entry' => [
      'id' => null,
      'media_type' => 'external-video',
      'position' => 10,
      'label' => 'TEST API',
      'types' => [],
      'disabled' => 0,
      'content' => [
          'base64_encoded_data' => base64_encode(file_get_contents('http://img.youtube.com/vi/axwE9q7llEQ/0.jpg')),
          'type' => 'image/jpeg',
          'name' => '0.jpg'
      ],
      'extension_attributes' => [
          'video_content' => [
              'media_type' => 'external-video',
              'video_provider' => '',
              'video_url' => 'https://www.youtube.com/watch?v=axwE9q7llEQ',
              'video_title' => 'keke',
              'video_description' => 'DESCRIPTION',
              'video_metadata' => ''
          ]
      ]
  ]
];

Image is saved but without any video content.

@bartlubbersen were you able to find some solution for that?

[bartlubbersen]

@rafal-kos

I did spend some time on this issue trying to debug it, but in the end I couldn't find the cause. But I do know the issue still exists.

[olysenko]

Hi, thank you for your report. Internal issue MAGETWO-61135 was created

[bartlubbersen]

Hi,

I did some more digging into this problem and found a sollution, I had to create a preference for the ProductRepository and a plugin for "Magento\Catalog\Model\Product\Gallery\Processor" to get it to work. So it should still be fixed by Magento, but maybe their developers or other people having the same problem van use it:

ProductRepository:
Replace in function processNewMediaGalleryEntry:

// Update additional fields that are still empty after addImage call
$this->getMediaGalleryProcessor()->updateImage(
    $product,
    $imageFileUri,
    [
        'label' => $newEntry['label'],
        'position' => $newEntry['position'],
        'disabled' => $newEntry['disabled'],
        'media_type' => $newEntry['media_type'],
    ]
);

With:

if (isset($newEntry['extension_attributes']['video_content'])){
    $newEntry = array_merge($newEntry['extension_attributes']['video_content'], $newEntry);
}

// Update additional fields that are still empty after addImage call
$this->getMediaGalleryProcessor()->updateImage(
    $product,
    $imageFileUri,
    $newEntry
);

Processor
Create a plugin with the following function:

public function aroundUpdateImage(
    Processor $subject,
    \Closure $proceed,
    \Magento\Catalog\Model\Product $product, 
    $file, 
    $data
) {
    $proceed($product, $file, $data);
    $fieldsMap = [
        'video_url' => 'video_url',
        'video_title' => 'video_title',
        'video_description' => 'video_description',
        'video_metadata' => 'video_metadata',
    ];
    
    $attrCode = $subject->getAttribute()->getAttributeCode();

    $mediaGalleryData = $product->getData($attrCode);
    
    if (!isset($mediaGalleryData['images']) || !is_array($mediaGalleryData['images'])) {
        return $this;
    }

    foreach ($mediaGalleryData['images'] as &$image) {
        if ($image['file'] == $file) {
            foreach ($fieldsMap as $mappedField => $realField) {
                if (isset($data[$mappedField])) {
                    $image[$realField] = $data[$mappedField];
                }
            }
        }
    }
    $product->setData($attrCode, $mediaGalleryData);
    
}

If you would prefer you could also create a preference for the Processor and just extend the $fieldsMap.
Hope it helps!

[cherreman]

Any updates on this? We're running in to the same issue.