Media 2.x includes admin UI to enable/disable media browser plugins (Upload/Web/Library/etc). This is usually configured at the field level, but there is a working patch over at #1434118: Make a Separate Config for WYSIWYG's Media Browser - see #17 and #28 to add an instance of the config form for the WYSIWYG editor plugin. It would be very useful to have this working for the CKEditor module as well. Assuming this patch is accepted by media's maintainers, is there a way to get the ckeditor module to pull the config from the wysiwyg_browser_plugins variable and use it for the ckeditor plugin as well?
I've been trying to find a way to do this myself but am a little stumped so far.
Comment | File | Size | Author |
---|---|---|---|
#11 | ckeditor-hook_into_media_admin-1649464-11.patch | 1.23 KB | Tess Bakker |
#9 | ckeditor-hook_into_media_admin-1649464-9.patch | 1.04 KB | kevinchampion |
Comments
Comment #1
mkesicki CreditAttribution: mkesicki commented@dafeder,
thank you for information. We try check this in future. Please be patient.
Comment #2
dafederNo problem. Happy to help if I can, let me know. I think this could actually be accomplished without having to submit a patch to the media module; the extra config options could just be added to the media settings form with a form_alter.
Comment #3
steveaylwin CreditAttribution: steveaylwin commentedHas there been any progress on this? We abandoned WYSIWYG + CKEditor library because of a cursor bug that required upgrade to v4 of CKEditor, and this was incompatible with that version of WYSI - we have since spent a lot of time integrating the standalone CKEditor module, which now means we don't have UI control of the media browser tabs (although new tabs can be added through media browser views). Obviously this could be done programmatically, but a UI option for enabling media browser tabs would be very useful.
Comment #4
renatovargas CreditAttribution: renatovargas commentedFrom inside CKEditor, we could use the code below. In the hook_form_FORM_ID_alter, we need to change the access of the WYSIWYG configuration to bypass "module_exists('wysiwyg')" test. In the hook_media_browser_params_alter, we load the parameters when a media browser is opened. The "allowed types" filter doesn't seem to work, but I think it's a Media's issue.
I didn't create a patch because I don't know if it's best to do this inside CKEditor or give support inside Media. There's already a Media's issue for the latter #1988792: Separate Config for CKEditor's Media Browser.
Someone that needs this configuration can use these hooks inside a custom module until one of the modules adds support to it.
Comment #5
aschmoe CreditAttribution: aschmoe commentedIn case anyone wants the patch mentioned in #4.
Comment #6
aschmoe CreditAttribution: aschmoe commentedComment #7
aschmoe CreditAttribution: aschmoe commentedForgot to "control H" my own module name out of that patch, sorry! :)
Comment #8
j3ll3nlThnx for this. Fixed it for me.
Comment #9
kevinchampion CreditAttribution: kevinchampion commentedRecent changes and refactoring in media, particular with regards to wysiwyg integration, changed these variable names from :
to:
Patch rerolled.
Comment #10
drclaw CreditAttribution: drclaw commentedFor anyone who has switched over to using the media_wysiwyg sub-module of media-7.x-2.x for ckeditor support, this issue might be of help? #2333855: Media Browser Settings for "WYSIWYG" aren't respected with CKEditor Module (e.g. browser tabs)
Comment #11
Tess BakkerHi,
Patch #9 works with the media-7.x-2.x-alpha3+114-dev, only it doesn't work when you use ckeditor with media and a file field with media on the same page.
The new patch will check for missing variables, that's always in the case of CKEditor.
Comment #12
shortspoken CreditAttribution: shortspoken commentedThanks @Tessa Bakker!
Patch #11 fixed it for me.
Media: 7.x-2.0-alpha4+26-dev
CKEditor: 7.x-1.16
Comment #13
junaidpvThanks! Patch #11 worked for me too
Media - 7.x-2.0-alpha3+98-dev
CKEditor - 7.x-1.16
Comment #14
jcisio CreditAttribution: jcisio at Axess Open Web Services commentedMedia integration has been removed.