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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

junedkazi’s picture

junedkazi CreditAttribution: junedkazi commented
FileSize
display-mode.png5.75 KB
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
CreditAttribution: Dave Reid 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 CreditAttribution: junedkazi commented
FileSize
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 CreditAttribution: 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
CreditAttribution: Dave Reid commented

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

junedkazi’s picture

junedkazi CreditAttribution: 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
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
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 CreditAttribution: aaron commented
Status: Needs review » Reviewed & tested by the community

This works really well.

junedkazi’s picture

junedkazi CreditAttribution: junedkazi commented

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

radiobuzzer’s picture

radiobuzzer CreditAttribution: 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 CreditAttribution: Devin Carlson commented
Status: Reviewed & tested by the community » Needs review
FileSize
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 CreditAttribution: 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 CreditAttribution: dureaghin commented

Committed #11 works for me. Thanks.