Posted by EugenMayer on April 30, 2011 at 12:33pm
7 followers
Jump to:
| Project: | Wysiwyg |
| Version: | 7.x-2.x-dev |
| Component: | Editor - CKEditor |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
As they slighly changed their header and size, the fgets 140 splits the version line in 2 pieces, what finally leads the regexp to not match.
Patch attached, highered lenght to 500.
| Attachment | Size |
|---|---|
| wysiwyg_ckeditor_version.patch | 508 bytes |
Comments
#1
Thanks!
I found no changes in the nightly build that would affect Wysiwyg so this has been committed to all branches.
The snapshots will be updated within 12 hrs and will then be able to detect CKEditor 3.5.4.
We should try to publish a new minor version before 3.5.4 is realeased, and check that the latest versions of the other editors are recognized as well.
#2
Well i will be checking the CKEditor on every release, so leave that one to me. i will report with fixes in the queue pretty early, if there are any. We are upgrading during one week on every release, rather earlier, and often testing the nighly builds due to all that issues with the FF / Safari stuff.
Just for the people reading this, when you are using ckeditor > 3.5.2 you will have issues with the Firefox browser, when you have something like tabs on the editor form (or a dialog or anything else "hiding" ( css, display:none ) the editor and showing it again). It comes down to the setting that disables the build-in CKEditor spell-checker ( disableWhatever .. ) .. Dont disable it and it will work again. There is no fix for this in CKeditor, not even in the nightly build.
#3
Great, and thanks for the info on the FF issue!
#4
Automatically closed -- issue fixed for 2 weeks with no activity.
#5
Just a note to say the fix for this is in 6.x-2.4 too.
#6
Same bug again
#7
Oups sorry my bad... I have two folder ckeditor/ckeditor . So it's just a bad error message but version detection works.