Support youtube shorts

I created a merge request that adds support for shorts, but also adds support for the url format youtube.com/watch/<id>, where before only youtube.com/watch?v=<id> was supported.

Applied patch and was able to enter YouTube shorts urls

Hi! I am facing the same issue with Drupal not finding a URL provider for youtube shorts videos and the suggested changes in the URL have not helped.
Could someone explain what applying patch means? Are there any other ways to deal with this issue?

Thanks!!

I am adding the changes found in the MR as a patch to be used by composer till it is merged.

@binnythomas To make your life easier in the future: you can add .diff at the end of the MR url to use it with composer as well (it will then automatically incorporate new changes made to the MR as well)

Hi @Jelle_S,

it will then automatically incorporate new changes made to the MR as well

That is exactly why I asked Binny to add the patch in here as the use of the .diff as https://git.drupalcode.org/project/video_embed_field/-/merge_requests/4.... is from what I know considered unsafe as it might change as you've described.

Such a change might result in break of automated build or so in your CI system.

I don't this this patch works for drupal 10.1. It still applies an accepts the shorts url. However, it won't actually play the video - instead the player directs you to view the video on YouTube.

you can try https://www.youtube.com/watch?v=xyz. xyz is the youtube video id which is the last part of the url. so you have to change https://www.youtube.com/shorts/xyz to https://www.youtube.com/watch?v=xyz. good luck

Works for me

Updated the tests.

I made a 3.x version of this issue: https://www.drupal.org/project/video_embed_field/issues/3509552 and just transfered MR and patch to be compatible with 3.x version.

Fixed in 3.0.x-dev branch.

This has been fixed in the 3.0.x branch in #3509552: Support youtube shorts 3.x. Let's make sure to include the unit test from that fix here too.