Hey Aaron,

It was awesome to see this released recently as a separate module.

I've been following your great work on the Media module (http://drupal.org/project/media) for Drupal 7... any plans to release a version of Media: Vimeo that supports Media in D7?

Thanks!

--Ben

Comments

aaron’s picture

yes, on the backburner for now...

Bevan’s picture

Does that mean Vimeo module will depend on Media module (and not emfield module)? What is involved in porting Vimeo to D7/Media? Are there any modules for other providers like Vimeo (e.g. YouTube) that can be used as an example for this?

Related: #957544: EmField for Drupal 7?

Bevan’s picture

Status: Active » Needs work
StatusFileSize
new4.8 KB

I started writing a Drupal 7 version of this that depends on Media Internet Providers module (in the Media package), based on the D7 branch of media_youtube. It is far from complete but is usable for some basic tasks, like adding a Vimeo video from the Media-picker by the URL of the Vimeo video page.

Bevan’s picture

I created the Vimeo Link Formatter module as a workaround for now. A number of code/features there will be re-usable in "Media: Vimeo" and we should merge these modules once "Media: Vimeo" is sufficiently mature.

steinmb’s picture

Hi @Bevan :) Did you ever get a answer to your question in #2?

Does that mean Vimeo module will depend on Media module (and not emfield module)? What is involved in porting Vimeo to D7/Media?

Bevan’s picture

Status: Needs work » Fixed

Kia ora Stein! (:

For this task you will create an upgrade path from the Drupal 6 version of Embedded Media Field (emfield) to the Drupal 7 version of Media.... For Drupal 7, Embedded Media Field's functionality is being replaced by the Media Module, which takes advantage of Drupal's new PHP stream wrapper support, however there currently is no upgrade path to move from Emfield to Media.

#988954: Create upgrade path from Embedded Media Field (6.x) to Media module (7.x)

However, if you can't wait for that and it's Vimeo implementation (and can't contribute to them), then Vimeo Link Formatter provides features that are reduced but comparable to Drupal 6 "Media: Vimeo" module, but for Drupal 7, and not taking advantage of (or depending upon) Media nor stream wrappers. PM me for the code.

Status: Fixed » Closed (fixed)

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

dmsmidt’s picture

Too bad to have a workaround module, would be nice if it worked with the Media module.

Subscribe

aaron’s picture

Status: Closed (fixed) » Active

support for media d7 is coming soon enough.

kmajzlik’s picture

subscribing

drupalok’s picture

subscribing

shinz83’s picture

subscribing

zberg007’s picture

subscribing

Jerome F’s picture

vasike’s picture

JGonzalez’s picture

subscribing as well

zkrebs’s picture

subscribing - i saw that the alternative way, using the vimeo link formatter, has a problem with views. how can one use the media module, and show the vimeo videos in views too - in d7?

perhenrik’s picture

subscribing

Gr3fweN’s picture

subscribing

jman77’s picture

subscribing

dpintats’s picture

Component: Code » Documentation

subscribing

so4’s picture

subscribing

rodych’s picture

subscribing

kmajzlik’s picture

for all subscribers: current hotfix = vimeo link formatter. works fine for me.

kreynen’s picture

I took a similar approach to @Bevan and just refactored media_youtube to work with peg.tv including the WYSIWYG support in http://drupal.org/project/media_pegtv. I need Vimeo support in Media and can put some time into media_vimeo. Would be helpful to have a D7 branch to work from.

@aaron any interest in opening the project up to additional co-maintainers for the D7 branch?

jerodfritz’s picture

subscribing

sebish’s picture

subscribing

midmood’s picture

subscribing

Jackinloadup’s picture

subscribe

aaron’s picture

thanks for the offer! i'm swamped these days, and rather than making everyone else suffer for it, i've added kreynen & Bevan as maintainers. you guys know the routine -- use issues & patches, and don't break anything if you can avoid it when committing, and please no broken releases...

thanks!

kreynen’s picture

Version: 6.x-1.0 » 7.x-1.x-dev
Status: Active » Needs work

Committed a quick refactor of the current media_youtube. Works for Video with iframe embed when js is enabled. Haven't tested beyond that. This is far from done. This is commit is only for developers looking to get involved so we can divvy up the work.

Hoping to talk to @Bevan tomorrow.

zkrebs’s picture

I tried #31 and it indeed shows my video! However, the thumbnail preview for Media gallery and on the node edit page do not show.

Bevan’s picture

Please remove me as a maintainer. I do not have time to contribute to this project sufficiently to justify being a maintainer. Thanks! :)

midmood’s picture

subscribe

Jean Gionet’s picture

Subscribing

kioko’s picture

subscribe

safetypin’s picture

#31 did not work for me, at all. After some investigation, it looks to me like the video id is not being properly communicated to the theme function.
Correction:#31 did work for me. Problem was that I didn't use the video URL, and instead used the embed code, which the comments explicitly say it can't handle. Oh well.

safetypin’s picture

StatusFileSize
new27.47 KB

This is my first attempt at a patch, so please be gentle. On my local machine, I'm seeing thumbnails pulled from Vimeo.

safetypin’s picture

StatusFileSize
new1.17 KB

I forgot that I changed another file, unrelated to this issue, so here's a cleaned up patch, for just the MediaVimeoStreamWrapper.inc file to make thumbnails work.

Jackinloadup’s picture

StatusFileSize
new1.17 KB

@idlewilder Awesome patch works great!

I modified #39 to pull in the large thumbnail so I could build high quality thumbnail grids for videos.

I figure if we pull in the higher quality images the D7 image style utility will re-size it the way we want.

Thanks for the great work!

zigmoo’s picture

Just tested #39. Works great for me, too. Kudos to @idlewilder!
We need to get this patch committed to the module, and released for general use.

zigmoo’s picture

Nice tweak, jackinloadup!

Jackinloadup’s picture

It would be be great if kreynen could commit this to the 7.x-dev.

What else is needed to close this issue? Do we need to pull in titles? or is that part of media_browser_plus?

kreynen’s picture

Status: Needs work » Closed (works as designed)

I committed the patch from #40.

http://drupalcode.org/project/media_vimeo.git/commit/8665a94

I've set the dev snapshot for the 7 branch to display.

I'm closing this issue. I don't think the Drupal 7 branch is "done", but I hate tracking issue with a über heading like this. Instead, open a new issue specific to the problem and track the status of a patch using "needs review" and "community reviewed" statuses.