A new CKEditor 4.0.1 is since today released.

I updated the editor and now i see in backend following error message:

Notice: Undefined offset: 2 in wysiwyg_ckeditor_version() (Line 85 from ckeditor.inc).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grinxols’s picture

Same problem here.
I've solved it replacing
return $version[1] . '.' . $version[2];
with
return $version[1];
in modules/wysiwyg/editors/ckeditor.inc line 85

Previously, I've done this too: http://drupal.org/node/1853550

pefferen’s picture

#1 helped, thanks, I created a patch

pefferen’s picture

Status: Active » Needs review
TwoD’s picture

Status: Needs review » Closed (duplicate)

Sorry, but that patch will break CKEditor 3.x versions, please note the comment above those lines.
I'll merge this into #1853550: Ckeditor 4.0 - The version of CKEditor could not be detected. since it's the same basic problem with all 4.x releases.

juliemariek’s picture

Thanks...#1 fixed my issue!

Belliger’s picture

#1 worked perfect for me. Thanks!

weeger’s picture

Using this version :

return $version[1] . ((isset($version[2])) ? '.' . $version[2] : '');
TGEink’s picture

So far #1 has worked for me also. Is there a concern and we should use your patch?

TGEink’s picture

I have used both the items -- the revisions to line 81 and 85 -- and no longer get any error messages about the installation. HOWEVER, I still cannot see the CKEditor. I get a kind of small grey bar so I can see that WYSIWYG is trying. With no results. I have tried this with Garland and Corolla with no joy from either. We do need this, so wondering what to do next. The site is http://tgeink.com.

TGEink’s picture

Priority: Normal » Major
Status: Closed (duplicate) » Active

I am changing this to active as I still cannot see the WYSIWYG CKeditor. I have no idea what to do next. Site is http://tgeink.com. We are using Garland. Tried with both Garland and Corolla.

I did use both the patches included in this thread, but no joy. They did allow me to see the editor installed without errors. I can select in the drop down menus for WYSIWYG for HTML or Text. I am getting a thin grey bar which makes me think it is trying... I am wondering if there is something in my CKeditor file that is wrong. I used the standard version. Uploaded as indicated. I originally tried to use a custom version, but when nothing worked went to standard, hoping that would fix.

TwoD’s picture

Priority: Major » Normal
Status: Active » Closed (duplicate)

#4 still applies. This is the wrong issue for this and the patches here are irrelevant as they do not fix many of the problems fixed in the other issue.
Working CKEditor 4 support is already onlyin 7.x-2.x-dev and will be part of the 7.x-2.3 release once we have a way to configure the new Advanced Content Filter it brings, or data loss will occurr. Again, do not use the "Drupal edition" of CKEditor, only the "Full" variant works.

GrantI’s picture

Worked perfectly for me - Drupal 7.23, WYSIWYG 7.x-2.2, CKEditor 4.2

trevorw’s picture

#7 solution worked for wysiwy 7.x-2.2 with ckeditor_standard_4.2

jay.lee.bio’s picture

#1 & #7 both work fine, although I think #7 is a better solution.

As for #9, having just moved over from CKEditor (the module) to Wysiwyg due to the Media module requiring it for one of its features, I went through what TGEink went through and found that with Wysiwyg you actually have to enable each button separately in the configuration page.

StijnStroobants’s picture

Issue summary: View changes
FileSize
531 bytes

Created a patch from solution #7

TwoD’s picture

Assigned: alexander.sibert » Unassigned

Please don't, the issue is marked a duplicate and this is not a solution. That patch WILL break things for you. See #11.