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

CommentFileSizeAuthor
#12 voicethread.zip1.75 KBmarcp
#5 voicethread.patch5.15 KBmarcp
#3 voicethread.zip1.7 KBbonobo

Comments

alex ua’s picture

alex ua commented

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...

bonobo’s picture

bonobo commented

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

bonobo’s picture

bonobo commented
Status: Active » Needs review
StatusFileSize
new voicethread.zip1.7 KB

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

bonobo’s picture

bonobo commented
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.

marcp’s picture

marcp commented
Status: Active » Needs review
StatusFileSize
new voicethread.patch5.15 KB

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.

alex ua’s picture

alex ua commented
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.

aaron’s picture

aaron commented

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

aaron’s picture

aaron commented

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

bonobo’s picture

bonobo commented

Did this ever get committed?

marcp’s picture

marcp commented

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?

marcp’s picture

marcp commented
Assigned: Unassigned » 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?

marcp’s picture

marcp commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new voicethread.zip1.75 KB

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.

bonobo’s picture

bonobo commented

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

Any feedback on this?

aaron’s picture

aaron commented
Status: Needs review » Fixed

added to both versions, thanks!

Status: Fixed » Closed (fixed)

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