Voicethread support

bonobo - July 31, 2008 - 21:10
Project:Embedded Media Field
Version:6.x-1.x-dev
Component:Embedded Video Field
Category:feature request
Priority:normal
Assigned:marcp
Status:closed
Description

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

#1

Alex UA - August 5, 2008 - 13:39

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

bonobo - August 5, 2008 - 15:32

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

bonobo - October 29, 2008 - 13:43
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

bonobo - October 30, 2008 - 00:42
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

marcp - October 30, 2008 - 03:25
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

Alex UA - November 19, 2008 - 10:48
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

aaron - November 19, 2008 - 22:01

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

#8

aaron - November 19, 2008 - 22:02

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

#9

bonobo - September 24, 2009 - 23:10

Did this ever get committed?

#10

marcp - September 26, 2009 - 15:13

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

marcp - September 28, 2009 - 17:41
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

marcp - September 28, 2009 - 23:11
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

bonobo - October 6, 2009 - 00:22

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

Any feedback on this?

#14

aaron - October 20, 2009 - 18:40
Status:needs review» fixed

added to both versions, thanks!

#15

System Message - November 3, 2009 - 18:50
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.