Implement a getName method in the VideoEmbedProvider

Created on 25 April 2023, over 1 year ago

Updated 12 May 2024, 7 months ago

Problem/Motivation

\Drupal\video_embed_brightcove\Plugin\video_embed_field\Provider\Brightcove does not currently implement a getName method.

This means that if you create a media entity attached to a field, it will end up with a default title from \Drupal\video_embed_field\ProviderPluginBase::getName

Steps to reproduce

1. enable a remote video media entity type to use brightcove
2. set up a field with a media reference field using a media browser
3. create content using the field

The title of the newly created media entity will be Brightcove Video (12345)

Proposed resolution

Implement getName so that the video title comes from the Brightcove metadata

See the video_embed_field module for an example.

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task

Status

Needs review

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom malcomio

Live updates comments and jobs are added and updated live.

Merge Requests

!2Implement a getName method in the VideoEmbedProvider
Open
🇬🇧United Kingdom malcomio
updated 6 months ago

Comments & Activities

Issue created by @malcomio
Comment over 1 year ago →
🇬🇧United Kingdom malcomio
Merge request !2Issue #3356220: Implement a getName method in the VideoEmbedProvider → (Open) created by malcomio
Status changed to Needs review over 1 year ago6:37am 30 April 2023
Comment over 1 year ago →
🇬🇧United Kingdom malcomio
As part of this merge request, it made sense to refactor getRemoteThumbnailUrl so that the two methods used a shared method to get the data.

Have also attached the MR as a patch.
Comment 7 months ago →
🇬🇧United Kingdom malcomio
Updated to include fetching thumbnail, as discussed on 📌 Implement a getName method in the VideoEmbedProvider Needs review

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024