This module does a great job of working with Youtube video URLs that use the 'v' query string parameter. However, YouTube also supports URLs with oembed like youtube.com/v/* and youtu.be/* , which simply use the URL path as the video ID rather than a query string parameter.

The YoutubeDurationGetterService could support these URLs too.

Patch to follow :-)

CommentFileSizeAuthor
#2 youtube_duration-other-schemes-3208887-2.patch590 bytesjames.williams

Comments

james.williams created an issue. See original summary.

james.williams’s picture

james.williams
Location Coventry, UK
commented
Assigned: james.williams » Unassigned
Status: Active » Needs review
StatusFileSize
new youtube_duration-other-schemes-3208887-2.patch590 bytes

Here's the patch!

I'm interested to see that this module has no releases, and doesn't seem to be maintained, despite being really quite good at fulfilling its use case. Would you consider making a release, or having a co-maintainer to help out? (I'd be happy to help! I maintain a variety of contrib projects already.)

stefan.korn’s picture

stefan.korn
Primary language German
Location Jossgrund
commented
Assigned: Unassigned » stefan.korn

@james.williams: Thanks for the patch. I will look into this in the next few days.

I will also do a release for the project. I should have done this already ... good that you reminded me.

stefan.korn’s picture

stefan.korn
Primary language German
Location Jossgrund
commented
Assigned: stefan.korn » Unassigned
Status: Needs review » Fixed

patch works perfectly, committed. Thanks again.

james.williams’s picture

james.williams
Location Coventry, UK
commented

Thank you! :-D

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.