jQuery adapter breaks save button

[Tim]

When using the jQuery adapter's .ckeditor() function, the save buttons of editors created in that way are broken. The button seems to work correctly, but the text that is submitted is whatever text was in the textarea before it was replaced by an editor.

A workaround is to do

CKEDITOR.replace($(element).attr('id'), opts)

but that kindof defeats the whole purpose of the jQuery adapter.

[Jakub Ś]

I'm kind of surprised that no one has reported this issue earlier.

Seems that toolbar Save button doesn't send updated contents (can be checked with Firebug in Net tab) to the editor but only the default text.
Just to compare, submit placed button under the editor works as expected.

Issue has been reproducible from CKEditor 3.1 (when jQuery adapter was introduced).

[Jakub Ś]

I just had a quick look at the patch and it seems to solve this particular issue.

I had hope that it will also solve #6906 but that would be to beautiful :

[Frederico Caldeira Knabben]

On the review, I started with the presupposition that there must be a reason to have that autoUpdateElementJquery hack there and so we should not simply remove it.

I've reached the original ticket for that (#4210), which doesn't clearly explains the reason but gives an idea for it.

In fact, after that, I was able to confirm that after patch editor.updateElement is being called twice when submitting the form. To test it, I've just added a alert( 'UPDATE!' ) call inside that function.

We must definitely avoid that.

[Piotr Jasiun]

This bug has been fixed during general jQuery adapter bug fixing #10281.