Closed (fixed)
Project:
Wysiwyg
Version:
7.x-2.2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
28 Jan 2015 at 09:46 UTC
Updated:
4 Dec 2016 at 12:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
googletorp commentedI've added a small patch for this.
Comment #2
googletorp commentedComment #3
twodJust out of curiosity, do you have an example of needing this hook?
My worry is that if the hook is added, another module will use it to try to support other versions/variants of some editor before that support is in Wysiwyg itself. Then once Wyswyg does support that version/variant it won't be in a way that is compatible with what the other module does, so the resulting behavior will be unpredictable.
Comment #4
jansete commentedIn some cases, you need to change urls for use CDN trought file_create_url.
We use ckeditor and for maximum performance our server admins don't allow static files with common urls, only works through CDN, this hookis very usefull for me.
I moved drupal_alter to the end, because some callbacks and config are defined after current drupal_alter position.
Comment #5
kristofferwiklund commentedLooking at the patch. The hook needs to be prefixed with wysiwyg module name. As it is now it conflicts with the Editor module (https://www.drupal.org/project/editor)
It should be named as hook_wysiwyg_editor_alter.
Otherwise we should enable hooks if it is needed. And if module update breaks custom alter hook it is up to the site administrator to handle it.
Comment #6
jansete commentedName changed!
Comment #8
twodThanks for helping with this!
I meant to set googletorp as the author as he made the original patch, but messed it up, sorry about that. Wish I could set more than one...
Comment #9
twod