Image plugin - Advanced Tab's Id field is not working

[Senthil]

Image Plugin - Advanced tab's Id field is not getting added to the editor area. Ex: I have added id as '12345' but the result in the editor source is missing the id field,

<img align="absbottom" alt="Oracle" border="10" height="41" hspace="11" src="/cache/abc.gif" vspace="33" width="145" /><br />

We have a business logic that is using this id field. Since the id field is not populated, our functionality is broken.Please fix it as early as possible.

[Martin Kou]

The bug comes from the insertHtml handler in the wysiwygarea plugin.

The insertHtml handler in wysiwygarea tries to insert the element into every range in the selection. Since there's the possibility of inserting multiple elements, the element is cloned and thus the id is stripped before insertion.

That does not make sense for dialogs that intend to only insert one element and would define an id for that element, however. So the insertHtml handler in wysiwygarea should be simplified to insert the element to the last range in the selection, and there should be no cloning.

[Frederico Caldeira Knabben]

This is actually a unique feature in V3. It's up to the end user to decide whether it's useful or not.

The problem with the proposed patch is that multiple selections are still selected after executing the function. It should at least delete the contents.

I have a quite simple proposal for it. I'm taking over the ticket so you can check it.

[Frederico Caldeira Knabben]

This new patch simply uses the original element for the first range, cloning it only if there is a multiple selection.

[Frederico Caldeira Knabben]

Fixed with [3244].