Add a hook to modify Media data

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Here is a re-roll for Drupal core >= 9.5.3.

Deprecation messages should be updated for 10.1 and removal in 11.0.

Still needs issue summary & title update

Tried to address the deprecated message update as per #71.

Update issue status because _utsavsharma seems to fix the borisson_ petition.
Working for us. Review and tested on Drupal 9.5.3

Patch failed to apply on D10 but I could make it work on D10 by copy/pasting same code at different line numbers in files. Also I could not find this test file in D10.1 :

/core/modules/media/tests/src/Kernel/ResourceFetcherTest.php

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rerolled patch #72 for 10.1.x branch, please review it.

Thanks!

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Tried to fix CCF for #76.

As per 🌱 Replace strpos/substr with PHP 8's str_starts_with() / str_contains() / str_ends_with() Active , we should replace strpos() !== FALSE with str_contains(). The patch on #78 uses twice strpos that should be replaced. Otherwise looks good to me.

added updated patch address #80

added patch against drupal 9.5.4

Still needs issue summary and title updates per #58 and #71

Reroll for 10.0.x

The #88 patch is working on my Drupal 10.1 project. Thanks!

Reroll for Drupal 11.x

After re-run the test on #90, all tests are passed now, Still needs to work on #86, So keep it same as 'Needs work'.

Upvoting a generic version of the hook: hook_media_source_metadata_alter. I currently have a use case for it, was reading through this thread getting hopeful that the patches would have gone that direction, but I see just the oembed specific hook has been created for now. I was able to install the patch as it is now, and the oembed hooks work, so thats cool, good work! They just don't work with non-oembed media items which is a bummer for my use case.

The latest version of the MR added changes to allow us to add the alter hook in MediaSourceBase::getMetadata. That still needs to be implemented in the MR though.

If we want to allow dynamic attributes to be mapped in the interface we also need to change getMetadataAttributes. The idea was to move the metadata attributes to the plugin annotation, and have getMetadataAttributes return whatever attributes are configured in the annotation. The annotations can then be altered via hook_media_source_info_alter.

That being said, technically you can already override the media source plugin class using hook_media_source_info_alter and override getMetadataAttributes and getMetadata for a specific media source. Which kind of makes me wonder if we need to implement this new hook at all. Although I know that lots of people probably find hooks much easier to implement than overriding a plugin class 🤔.

Re-rolled patch 90 for use on core 10.6.1

I'm assuming you meant 10.1.6 because 10.6.x doesn't exist yet. 😉

Ambient.Impact → changed the visibility of the branch 11.x to hidden.

Created a new branch against 11.x because trying to rebase the old one onto it was a nightmare; reapplied the latest patch, merging in core changes as needed; also hiding the existing patches - download the patch from GitLab diff pls.

@Ambient.Impact I may be missing something but I cant seem to download a patch for your branch. I think you need to create a MR for it?

Based on the suggestion made on #86, I am adapting the title and description of the issue.

Pending to implements hook "hook_media_source_metadata_alter" on MR https://git.drupalcode.org/project/drupal/-/merge_requests/1614 add tests coverage and API documentation.

Upload loze's patch from #99 in order to use it with composer patches.