CKEditor initialisation throws a Javascript error in IE6-IE8 when processing the teaser of a drupal content.
In IE9 beta it seems to work.
Problem is located at ckeditor.utils.js on Lines 41 and 130.
Creating the global var teaser like this:

if (teaser = Drupal.ckeditorTeaserInfo(textarea_id))

does not work in older IE's.

I fixed this localy by defining the teaser var directly like this:

var teaser = Drupal.ckeditorTeaserInfo(textarea_id);
if (teaser)

This also has the Side effect in keeping the teaser var local.

The Fix seems to work fine.j

Comments

dczepierga’s picture

dczepierga CreditAttribution: dczepierga commented
Status: Active » Fixed

I fix it and commit to GIT.

Please check the latest dev release and let me know if you notice any poblems with it. Remember to clear Drupal cache and browsers cache before testing.

tobey_p’s picture

tobey_p CreditAttribution: tobey_p commented

I checked the latest Dev and the Fix is missing for the Function Drupal.ckeditorOff.
There is still an instance of 

if (teaser = Drupal.ckeditorTeaserInfo(textarea_id)) {

So when you switch back to plaintext it still breaks in IE6-IE8.

Cheers,
tobey

tobey_p’s picture

tobey_p CreditAttribution: tobey_p commented
Status: Fixed » Needs work

Just changed the Status to "needs work".
Hope that is o.k.

Cheers,
tobey

dczepierga’s picture

dczepierga CreditAttribution: dczepierga commented
Status: Needs work » Fixed

I fix it also there - look here for diff: http://drupalcode.org/project/ckeditor.git/blobdiff/98e98281779d4ce82c6c...

Sometimes is more time needed to Drupal rebuild DEV package.

Greetings

tobey_p’s picture

tobey_p CreditAttribution: tobey_p commented

Ah ok, didnt know about that. The Diff looks indeed good.
So i downloaded the dev Package in the middle of rebuilding :-)

Cheers,
tobey

Status: Fixed » Closed (fixed)
Issue tags: -JavaScript, -ie7, -ie6, -ie8

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

Issue tags: +JavaScript, +ie7, +ie6, +ie8

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.