At the moment the file entity title is the video id - it would be great to use the video title instead.

For media_youtube a patch is available: http://drupal.org/node/1368714

Based on this patch I have created a similar one for media_vimeo. It required to update the oembed API call a little bit - it seems that the current code is not working anymore.

Patch follows..

CommentFileSizeAuthor
#6 media_vimeo_1422980.patch7.48 KBdrewish
#1 media_vimeo-set_title_of_file_entity-1422980-2.patch1.63 KBvalderama

Comments

valderama’s picture

valderama commented
Status: Active » Needs review
StatusFileSize
new media_vimeo-set_title_of_file_entity-1422980-2.patch1.63 KB

Here's the small patch.

KoCo’s picture

KoCo commented

works for me so far
thx

valderama’s picture

valderama commented

the corresponding patch for media: youtube was comitted by dave reid, probably the maintainer of media: vimeo can do the same?

see: http://drupal.org/node/1368714

paulmckibben’s picture

paulmckibben
he/him
Primary language English
Location Atlanta, GA
commented

Looks good!

I applied this patch without any issue. I verified that the title of the file entity is now set to the title of the Vimeo video, both when I use file/add/web, and when I use a file field + media selector on a node. Thanks for the good work!

paulmckibben’s picture

paulmckibben
he/him
Primary language English
Location Atlanta, GA
commented
Status: Needs review » Reviewed & tested by the community

We have two good reviews, no issues raised. I'm going to mark it RTBC.

drewish’s picture

drewish commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new media_vimeo_1422980.patch7.48 KB

Okay did some work to sort this out and clean up the streamwrapper and vimeo handler. I fixed the URLs we're generating so drupal_realpath works again. Then used the API call change valderama's pointed out.

drewish’s picture

drewish commented
Status: Needs review » Fixed

Aaron gave me commit access so I've pushed this fix up.

steinmb’s picture

steinmb commented

Works great! Thanx :)

Status: Fixed » Closed (fixed)

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