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.
The WYSIWYG module did not detect CKEditor 4.3 or 4.1.3 versions. I did manage to get CKEditor 3.6.6.1 to work. Forgive me if this has already been submitted. I searched but was unable to find anything already. This is my first report. It does not seem like a bug so I classified this as a request to update support for v4.3. Thanks!
Comment | File | Size | Author |
---|---|---|---|
#22 | wysiwyg-ckeditor-not-detecting-2141117-22.diff | 774 bytes | emanaton |
#16 | ckeditor_4.3_toolbar_buttons_messed_up.png | 5.38 KB | klonos |
#12 | wysiwyg-ckeditor-not-detecting-2141117-2.diff | 826 bytes | dasrecht |
#9 | wysiwyg-ckeditor-not-detecting-2141117.patch | 729 bytes | Perignon |
Comments
Comment #1
cmanalansan CreditAttribution: cmanalansan commentedHow to get CKEditor 4.x working with the Drupal 7 WYSIWYG module:
http://www.webopius.com/content/717/how-to-get-ckeditor-4-x-working-with...
Comment #2
Starminder CreditAttribution: Starminder commentedFollowed the fix in #1, I'm still stuck. Is there a trick to get WYSIWYG to refresh and check again?
Comment #3
MacMladen CreditAttribution: MacMladen commentedYou do not need any fix, just use dev version that recognize CKEditor 4.x directly and works fine.
Comment #4
tsvenson CreditAttribution: tsvenson commented"just use dev version" is not a very good advice for production sites to be honest.
CKEditor 4.x has been out for quite some time and it would be good it this module could be at least updated to recognize the currently recommended releases.
Comment #5
MacMladen CreditAttribution: MacMladen commented@tsvenson I believe you are right, but I prefer
dev
topatch
version and also from experience, I know that there are other reasons why developers are waiting for next release. Thatdev
comes from some patch and is validated by developers as working solution that (presumably) is working while waiting for some other things for new release (do not have to be CKE 4 related at all).On the other hand, this works, it is not too complex to check if there are some undesired implications and one can always stick with version that works for them.
I've been using it for some time and it seems fine to me.
Sure, not every
dev
release can follow the same advice.Comment #6
tsvenson CreditAttribution: tsvenson commented@MacMladen:
I think you might have misunderstood my comment a little. This isn't about if a working version is available in the dev release or as a patch. It's about that this module is dependent on third party libraries. Plus the fact that this module is useless without any of those third party libraries.
Thus, to be able to keep up to speed with those third party libraries there is a need for different release strategies. One for bug/security fixes, one for new functionality and one to keep up to speed with new releases of these third party libraries.
It is also a known problem that newer versions of some of the third party libraries, especially CKEditor, is not recognized by this module.
Also, those new third party library release can very well contain security fixes too.
The latest full release of the Drupal 7 version of this module was over 13 months ago. Thus, it is quite easy to draw the conclusion there is now strategy for keeping up to speed with third party libraries.
A dev version contains work in progress, new features that aren't fully tested and so on. Thus, again, just saying to user to use it on live production sites is not the way to do it.
Comment #7
jay.lee.bio CreditAttribution: jay.lee.bio commentedThank you, #1 worked for me. I did have a resulting error, which was quickly fixed by https://drupal.org/node/1883456 (#14).
Comment #8
baram204 CreditAttribution: baram204 commented#1 working, Can't understand why this was not resolved or patched.
and.. #1 patch will cause "Notice: Undefined offset: 2 in wysiwyg_ckeditor_version() (Line 85 from ckeditor.inc)." error.
#7 will resolve your second error. cheers!!
Comment #9
Perignon CreditAttribution: Perignon commentedTime this was patched. Been going on way too long.
Comment #10
baram204 CreditAttribution: baram204 commentedThanks Perignon.
Comment #11
gilgabar CreditAttribution: gilgabar commentedThe patch in #9 works well.
Comment #12
dasrecht CreditAttribution: dasrecht commentedSeems like the patch needs some update. did not work for me with 7.x-2.2
see updated patch.
Comment #13
Perignon CreditAttribution: Perignon commented@dasrecht This issue isn't against 7.x-2.2. This is against 7.x-2.x-dev. The code was changed between 2.2 and 2.x-dev, so hence why my patch would not work for 2.2 :-D.
See here: http://drupalcode.org/project/wysiwyg.git/history/ee64524b3edb2aa21098a2...
Version 2.2 of the CKeditor include was changed over a year ago. The Dev branch has several changes.
Comment #14
dasrecht CreditAttribution: dasrecht commentedEww, I am sorry my misstake :-|
Just ran into the issue today and tought to roll a patch.
Comment #15
Perignon CreditAttribution: Perignon commentedI was running 2.2 like you. I switched to dev to fix a lot of the little nit-noid issues with WYSIWYG
Comment #16
klonosI updated to 4.3.1 from 4.1.2 and the latest WYSIWYG 7.x-2.x-dev picked it up without any patch applied. The only issue I'm having is that the buttons in the toolbar are misaligned/messed up. They do work though, so this is merely a display issue. Anybody else seeing this (perhaps know of an issue filed for it - I searched but could not find any)?:
Comment #17
Perignon CreditAttribution: Perignon commentedWhat would be a JS problem. Have you attempted to do things like: clear cache of browser completely, try another browser.... typical stuff like that.
Comment #18
klonosI do admin work from firefox and test both there as an admin + on chrome as an authenticated user. The issue appears in both browsers even after clearing all caches from firefox as admin. Just to be sure I created a new vanilla profile in firefox and tested there and indeed the issue is not present.
My problem though is that I need this fixed in my current working profile as well where clearing all browser cache is not an option. Asking all of the site's authors to do the same is not an option either. Shouldn't a full (Drupal) cache clear solve this in theory? What other options do I have?
PS: I realize now that this is OT here, honestly sorry for that - I should know better, but too late now. If this takes more than a couple of more comments I promise to take it elsewhere.
Comment #19
klonos...ok one last comment and that's it: What finally did the trick was adding a button (I randomly chose to use the "About" button) to the WYSIWYG profile, saving, then refreshing the edit page and then finally removing the added button and saving once again. That seems to have forced the browser-side cache to be refreshed. It initially worked only for firefox. Chrome required to also remove and add back an existing button in order for this magic to work.
Anyways, perhaps all this needs to be documented someplace or better if one knows of a way to actually achieve the same thing with code (some kind of custom, CKEditor-targeted cache clearing), then it should be implemented in order to spare users from the trouble I went through.
I'm now shutting up and once again sorry for the noise.
Comment #20
klonos...in #16 I said:
This might have got lost with all the comments about the messed up buttons, so I wanted to be clear that I have tested this in another couple of websites and 4.3.1 is detected by the latest dev of WYSIWYG without having to apply any patches. So, is this issue here still valid?
Comment #21
TwoDNo, this issue is not valid. Initial CKEditor 4 support was committed long ago, and a few tweaks have been applied since.
I'm currently trying to clean up, and catch up with, the issue queue so I can focus on just the issues which need to - or are small enough - to be included before there is a new release.
Comment #22
emanaton CreditAttribution: emanaton commentedRe-rolling patch from #12 for it to be make-file friendly.
Comment #23
TwoD@emanaton, please don't reroll this anymore. That patch is far from enough to make CKEditor 4 actually work.
It's all in Wysiwyg 7.x-2.x-dev now.