It needs to read more characters with the new version

CommentFileSizeAuthor
#7 wysiwyg-ckeditor-3.5.4-7.patch507 bytesedmund.kwok
#1 wysiwyg-ckeditor-1154434-1.patch508 bytesmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Needs review
FileSize
wysiwyg-ckeditor-1154434-1.patch508 bytes

http://ckeditor.com/download/releases

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Fixed

Already fixed in latest dev snapshot.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented

ah I patched against 6.x-3.x Need to be more careful when selecting the version from here http://drupal.org/node/181465/git-instructions

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented

oy, thanks for the pointer! I'll check with TwoD whether we might be able to remove those orphan 3.x branches.

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented

Yes we can get rid of them. Think we discussed that earlier and said to branch of 2.x once 3.x gains momentum in/as master.
That should avoid confusions like this one.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

edmund.kwok’s picture

edmund.kwok CreditAttribution: edmund.kwok commented
Version: 6.x-2.x-dev » 7.x-2.x-dev
Status: Closed (fixed) » Needs review
FileSize
wysiwyg-ckeditor-3.5.4-7.patch507 bytes

Patch required for 7.x-2.x dev too :)

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented
Status: Needs review » Closed (fixed)

This is still already fixed, see #1143104: Newer ( 3.6.3 ) CKeditor version not detected by ckeditor.inc and commit 26b48879a6d712068cd3f085dec8d8182304887f.

mv_alex’s picture

mv_alex CreditAttribution: mv_alex commented

But the error still occurs with the latest releases.

I got it on core 7.2, Wysiwyg 7.x-2.0 with CKEditor 3.6.

RedTop’s picture

RedTop CreditAttribution: RedTop commented

Same issue here. Core 7.2, Wysiwyg 7.x-2.0 with CKEditor 3.6.1

(subscribing)

s.Daniel’s picture

s.Daniel CreditAttribution: s.Daniel commented

wysiwyg D6 dev works with 3.6.1

From a quick look at the code of D7 dev version it should also. Have you tested the dev version?
Fixes will always first be applied to dev and then shipped with the next release.

Therefore the status was set back as it is fixed in the dev version.

mv_alex’s picture

mv_alex CreditAttribution: mv_alex commented

After updating to 7.x-2.1 it now works with CKEditor 3.6.0.6902.

Thanks!