Closed (fixed)
Project:
Media: Dailymotion
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Nov 2012 at 16:50 UTC
Updated:
27 Oct 2013 at 05:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
leelooch commentedThanks for the patch !
It worked well for me on media 7.x-2.0-unstable7 though I think it could be difficult for newbies to guess they have to check Dailymotion Stream in admin/structure/file-types/manage/video/edit page.
Comment #2
oystercrackher commentedPatch worked great, but I still had to add video/dailymotion mimetype and select Dailymotion stream in admin/structure/file-types/manage/video/edit page. Also had to clear the caches.
Thanks
Melissa
Comment #3
leelooch commentedhere's a copy of the code added in to the media_youtube.install file, maybe it could be useful to add this to :
Comment #4
dave reidThe update function in #3 is no longer necessary due to #1949520: Remove media_youtube_update_7202() and media_youtube_file_default_types_alter()
Comment #5
bacarndiaye commentedThank very mutch, it's helpfull
Comment #6
bacarndiaye commentedComment #7
garphyI'm reopening this as it hasn't got any status/feedback from a maintainer.
Comment #8
garphy@Rob_C, your patch doesn't only change function location (hook_media_internet_providers) but also alter what's this hook returns.
I don't think this alterations are mandatory for this particular issue so I would suggest you open another issue for this one.
I rerolled the patch to include only the addition of the required hooks, making it more suitable for inclusion/commit.
(Side Note: why do you removed the 'image' key in hook_media_internet_providers implementation ?)
Comment #9
marvil07 commented@Rob_C, @garphy, thanks for the patches.
@David Reid, thanks for the hint!
I have just followed David suggestion and removed the
hook_file_default_types_alter()from the last patch and added to 7.x-1.x.