After #1748952: Add simple HTML5 file field formatters for <audio> and <video> and in alpha2 release only I see this issue. IF I revert to alpha 1 it works fine.

SO in alpha 2 it is trying to run the internet source like youtube/vimeo etc through the video tag and video tag does not support "video/youtube" as mime source. Same is the case with vimeo.

I am attaching some screenshots as well.

CommentFileSizeAuthor
#11 2073001-11-file-displays-weight.patch27 KBdevin carlson
#7 2073001-file-displays-weight.patch13.27 KBdave reid
#3 file-entity-video-display.png15.79 KBjunedkazi
#1 display-mode.png5.75 KBjunedkazi
#1 display-mode.png5.75 KBjunedkazi
player.png6.56 KBjunedkazi
html.png4.47 KBjunedkazi
firebug-error.png8.67 KBjunedkazi

Comments

junedkazi’s picture

junedkazi commented
StatusFileSize
new display-mode.png5.75 KB
new display-mode.png5.75 KB

Attaching field settings and display mode

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

In your 'Manage file display' settings for the Video file type, you should order the file formatters for Youtube or your more specific types of remote media *higher* than than the Video formatter. We might need to check the default settings of Media: Youtue or oEmbed to ensure their default configs follow this behavior.

junedkazi’s picture

junedkazi commented
StatusFileSize
new file-entity-video-display.png15.79 KB

So on further debugging I noticed that the display order was causing the issue. SO if I move video right at the bottom then the internet source videos work fine and load in an iframe. But if I keep it right at the top then it tries to load in hrml5 format.

I believe we should have some kind of a check to check the source/mime type.

junedkazi’s picture

junedkazi commented

@Dave you were very quick. Before I could post my next response you had the same answer. So does this qualify as a bug ?

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

@junedkazi: Was this a fresh install of both Media and Media: Youtube, or an upgrade from 1.x?

junedkazi’s picture

junedkazi commented

I have been running dev version of media/file entity/youtube for a while. But after the alpha release I decided to move to alpha release as it is much more stable.

Currently I am running

Media 7.x-2.0-alpha1
Media: YouTube 7.x-2.0-rc3+8-dev
File entity 7.x-2.0-alpha2

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 2073001-file-displays-weight.patch13.27 KB

This should at least fix the default ordering errors by putting the Audio and Video file formatters *last* for the relevant file types. Also removes some un-relevant file formatters for some file types as well (no export for audio and video on document file types).

aaron’s picture

aaron commented
Status: Needs review » Reviewed & tested by the community

This works really well.

junedkazi’s picture

junedkazi commented

It works perfectly fine. SO RTBC from me as well.

radiobuzzer’s picture

radiobuzzer commented

I confirm that this patch works. Fix should be given a priority as it breaks the recommended version alpha2 as well.

devin carlson’s picture

devin carlson commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 2073001-11-file-displays-weight.patch27 KB

#7 works as advertised but I think that I would go as far as removing all of the un-relevant file formatters and ordering all of the relevant formatters last.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

Committed #11 to File entity 7.x-2.x after testing it again to confirm that it fixes the issue as the RTBC patch in #7 did while removing the remaining irrelevant formatters.

Status: Fixed » Closed (fixed)

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

dureaghin’s picture

dureaghin commented

Committed #11 works for me. Thanks.