Project:Embedded Media Field
Version:6.x-1.x-dev
Component:Embedded Video Field
Category:feature request
Priority:normal
Assigned:marcp
Status:closed (fixed)

Issue Summary

Hello, all,

I was wondering if there are any plans for supporting Voicethread (voicethread.com) as a provider --

It currently works if you use the "custom url" provider, and then pull the src uri from the embed code as the path to the file -- however, this is beyond the reach of many end users.

Are there any plans to build out voicethread support? If they are on the radar, that'd be great; if not, we might see about getting some support to contribute this. We're pretty full out with ongoing work, and don't have anything concrete on the table, but we have seen some interest in this of late.

So, if someone is working on this, let's talk -- and if no one is working on this, that's good to know to :)

Thanks,

Bill

Comments

#1

Hey Bill-

There are no active plans on developing voicethread support, as most of the energy being put into the module right now are centering around the Drupal 6 port (aaron has been working hard on media stuff for drupal 7 and a bunch of other new media modules), and honestly I hadn't even heard of voicethread before.

So, I suppose the answer is: if you can make it happen, go for it! I'll definitely commit it if you do...

#2

Hello, Alex,

Sounds good -- we've actually been talking with the folks behind voicethread, and they might be coding this up themselves -- I've pointed them at this thread, and at the existing .inc files as models.

Thanks for getting back to me, and for being open to the addition.

Cheers,

Bill

#3

Status:active» needs review

Hello, Alex,

The folks at Voicethread coded this up. We have tested it on a couple sites in D5, and all works as it should. It requires the full embed code, as opposed to the URL as with (for example) YouTube.

I'll set the status to code needs review, but this has been working very nicely for us.

Cheers,

Bill

*edit* -- to make this work, download the zip file to get "voicethread.inc" -- place this file in the contrib/video_cck/providers/ directory, and configure as you would any other provider

AttachmentSize
voicethread.zip 1.7 KB

#4

Status:needs review» active

We have a reworked .inc file coming for this that will work with the url of the video, instead of requiring the embed code. We'll get it uploaded later today or early tomorrow.

Setting back to active.

#5

Status:active» needs review

Here's the patch that Bill mentioned. It applies against 5.x-1.2. Put the file in emfield/video_cck/providers and run:

patch < voicethread.patch

It should just create a new voicethread.inc file that is similar to the one included in the zip file in #3 with the improvement that URLs like http://voicethread.com/share/237122/ will now work. The code's also been cleaned up a bit based on Coder's recommendations.

I haven't played around with voicethread enough, but it looks like there may be other URL patterns that we'll want to match.

AttachmentSize
voicethread.patch 5.15 KB

#6

Version:5.x-1.2» 5.x-1.x-dev
Status:needs review» patch (to be ported)

I'll talk with Aaron about this today, but we should be committing this to the 5.x dev version soon and will then work to port it to Drupal 6.

#7

if it works for 5, it should work as is for 6 as well.

#8

umm, no, sorry, forgot about the module name change...

#9

Did this ever get committed?

#10

Looks like it's not in either 5.x or 6.x.

If I re-roll for both versions would there be a chance of getting this into the next releases?

#11

Assigned to:Anonymous» marcp

Assigning this to myself.

Question for the maintainers - would a working patch have a chance of being accepted or should we create a new project for Voicethread support?

#12

Version:5.x-1.x-dev» 6.x-1.x-dev
Status:patch (to be ported)» needs review

Attached is a zipped 6.x version of just the new voicethread.inc file. It appears to be working for me. I changed all video_cck occurrences to emvideo and added a proper emvideo_voicethread_emfield_subtheme() function.

AttachmentSize
voicethread.zip 1.75 KB

#13

This has worked well in testing on a couple different sites.

Any feedback on this?

#14

Status:needs review» fixed

added to both versions, thanks!

#15

Status:fixed» closed (fixed)

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

nobody click here