This patch creates the required mime types. (It also moves 1 function up, almost all media provider modules have a similar order, minor detail, but nice enough.)
I have it working on the current media 7.x-2.x git version.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | media_dailymotion-create_mime_type-1850924-8.patch | 842 bytes | garphy |
| media_dailymotion_mime_types_patch.patch | 1.8 KB | rob c |
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.