The attached patch should fix two issues which seem to have been introduced recently in the JS code in fckeditor.popup.html.

1. A leftover debug line was preventing content from being transferred from the editor to the Body field when "OK" was clicked in FF and IE at least.
2. Editing block content wasn't working because the editor was trying to put text in a non-existent teaser field.

CommentFileSizeAuthor
fckeditor-bugs.patch1.69 KBGarrett Albright
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jorrit’s picture

Assigned: Unassigned » Jorrit

You're absolutely right about the console.log. I didn't check for teaser === null, because I only attach the teaser GET parameter when there is a teaser. But checking again doesn't hurt, I think. Is the === comparison also valid in JS? I thought it was only valid in PHP.

Garrett Albright’s picture

Sure it is. Try running 1 == '1' and 1 === '1' in the Firebug console or similar and you'll see it behaves as it does in PHP.

Jorrit’s picture

I have committed a fix, slightly different than your patch. You will be able to find it in the next -dev release, which are built nightly. I hope you like it.

Jorrit’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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