Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: bacarndiaye commentedThank very mutch, it's helpfull
Comment #6
bacarndiaye CreditAttribution: bacarndiaye commentedComment #7
garphy CreditAttribution: garphy commentedI'm reopening this as it hasn't got any status/feedback from a maintainer.
Comment #8
garphy CreditAttribution: garphy commented@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 CreditAttribution: 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.