The following patch is a direct backport of the work I am doing with Drupal 8 found here http://drupal.org/node/1174892.

I would like for the Media module to be the official D7 backport of this work if you guys are willing. If we take this, then I can officially deprecate the http://drupal.org/project/html5_media module, which would be sweet.

Patch to follow.

CommentFileSizeAuthor
#2 media-video_audio_support-1496528-2.patch369.36 KBtravist
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Awesome. I know for sure we at least want to have <audio> and <video> native tag support in Media. Maybe we add some simple support first but then I would love for us to be able to support a fully-featured player included in Media. It makes total sense to do.

travist’s picture

travist’s picture

Status: Active » Needs review
dmurkerson’s picture

Hello:

I've just tried to apply this patch to the 2.0 dev version of the Media module on my Drupal 7 site according to these instructions: http://drupal.org/patch/apply and I'm getting "Hunk Failed" errors.

I'm trying to incorporate the Media module into Wysiwyg so that I can upload media files and have them display in a player on my site. This patch was listed on the Media main project page as an update to be able to do this.

Any help would be appreciated.

Thanks,
Danielle

tsvenson’s picture

@dmurkerson: Hunk failed means that the patch couldn't be applied cleanly. This usually is due to that since the patch was created, a lot of other code has been committed to the project, casing this patch the be "out-of-sync" so to speak.

To fix it you need to manually apply the patch. Alternatively wait until someone re-rolls the patch against a current checkout of the code.

dmurkerson’s picture

Yes, that is why I commented on this. I began to apply it manually (I'm no stranger to code) however, since there were so many files this patch was fixing/creating I thought it best to use the patch functionality in Linux to do it for me so that there would be less chance I'd miss something in applying it manually.

I'll keep watch here and elsewhere as well as apply it myself again and see if that helps.

Thanks.

travist’s picture

I will work on getting an updated patch and player.

dddave’s picture

jfyi I linked to this issue in http://groups.drupal.org/node/233238

pschuelke’s picture

Status: Needs review » Needs work

The patch does indeed have hunk errors when applying.
I just tried to do a manual install of the #2 patch with 2.x dev. It doesn't seem to like to play with the most recent changes.

pschuelke’s picture

There is a patch that does this here.

Devin Carlson’s picture

Status: Needs work » Closed (duplicate)

As mentioned in #10, this issue belongs to file entity.

#1748952: Add simple HTML5 file field formatters for <audio> and <video>