Unable to embed Flickr Photosets [#1847060]

Hi,

With 7.x-2.0-unstable6, media_flickr-7.x-1.0-alpha3, wysiwyg-7.x-2.2, and CKEditor 3.6.5.7647, I'm able to embed flickr photoset using URL as videos.

With 7.x-2.0-unstable7, I'm now unable to embed a flickr photoset. Media modules reports "Only the following types of files are allowed to be uploaded: video."

I've add the 'application/x-shockwave-flash' mimetype to the mimetype list for the video file type, but it is still not being validated by media_file_validate_types().

Any ideas? Jason.

Comment	File	Size	Author
#3	media_flickr-mimetypes-1847060-3.patch	728 bytes	dwatts3624

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

dwatts3624 CreditAttribution: dwatts3624 commented 8 December 2012 at 22:26

I'm having the same issue with the module. Based on issues I've run into with other media modules I'm guessing you're onto the root of the issue with the mimetype.

Comment #2

dwatts3624 CreditAttribution: dwatts3624 commented 8 December 2012 at 23:00

I looked through the code a bit. I don't have the skills to solve this but it seems like the module will set the mimetype video/flickr if it detects a photoset. Since this doesn't seem to be added and isn't even an available type on the video file type settings I assume that's what's preventing this from working.

Comment #3

dwatts3624 CreditAttribution: dwatts3624 commented 8 December 2012 at 23:36

Version:	7.x-1.0-alpha3	» 7.x-2.x-dev
Status:	Active	» Patch (to be ported)

File	Size
media_flickr-mimetypes-1847060-3.patch	728 bytes

I guess this was easier than I thought since it's already been discussed in the context of youtube and vimeo (#1823078: Create video/vimeo mimetype). I've attached a patch for testing here. Everything works on my end.

Comment #4

bkat CreditAttribution: bkat commented 12 December 2012 at 01:08

You also need to the following sql to make existing flickr content render properly
update file_managed set type='video' where type='undefined' and filemime='video/flickr';

Comment #5

jasonawant

he/him

English

New Orleans, USA

CreditAttribution: jasonawant commented 22 January 2013 at 17:47

@dwatts3624

Tested patch. Works here too, thanks!

Are you able to see the preview thumbnail of photosets?

Jason

Comment #6

sanduhrs

he/him

German

🇪🇺 Heidelberg, Germany, Europe

CreditAttribution: sanduhrs commented 28 January 2013 at 15:33

Component:	Miscellaneous	» Code
Category:	support	» task
Status:	Patch (to be ported)	» Reviewed & tested by the community

#3 Works as advertised.

Comment #7

aaron CreditAttribution: aaron commented 2 April 2014 at 17:41

Issue summary:	View changes
Status:	Reviewed & tested by the community	» Needs work

The patch at #3 no longer applies.

Comment #8

steinmb CreditAttribution: steinmb commented 19 June 2014 at 22:18

Status:

Needs work

» Fixed

Patch in #3 is no longer needed. Just checked out from git and did a clean install to test the state of this module:

file_entity
media
media_flickr
Drupal 7.28

Foto set works just fine. All you need to do is go to admin/structure/file-types/manage/video/file-display and check 'Flickr Photoset ' and it just works. Single images is configured/enabled in admin/structure/file-types/manage/image/file-display. Also tested adding a combination of slideshow, single image and locally stored image in the same multivalue field on a single node and that also worked just fine.

There might be migration issues moving from 7.x-1.x to 7.x-2.x but that need to be addressed in a separate issue. Have not checked what happen if you run on a SSL enabled site, perhaps the slideshow get blocked by the browser if it runs in mixed mode.