[IE] : Can't add link to selected text

[Jakub Ś]

In IE from CKEditor 3.6.2 it has been impossible to change selected text into link (Link is added next to text).

Steps to reproduce:

1. Open ​http://www.ckeditor.com/demo
2. Click the 'Code' button
3. Replace all code by this HTML code:

<table><tr><td>test test test test</td></tr></table>

4. Click the 'Code' button again
5. Double-click the 2nd, 3rd or 4th word 'test' (if you select the first one, the problem doesn't occur).
6. Click the Add Link button
7. URL: google.com
8. Click OK

Expected result:
The selected word is now a link to google.com

Actual result:
The selected word is not a link. The link to google.com is added in front of the selected word.

NOTE:If you undo (CTRL-Z) after step 8, and repeat step 5 t/m 8, the link is created as expected. So it seems to be a problem only the first time.

Issue has been reproducible in IE browsers from CKEditor 3.6.2 rev [7243].

[Jakub Ś]

This may be a duplicate of #8475

[Jakub Ś]

#8546 was marked as duplicate

Another TC from the dulicate:

1. Paste in the below code

   <ol> 
   <li id="uavklart">Pri 1: Utdanning.no: Vurdere om vi hvilket innhold&nbsp;vi ønsker å levere og hvordan - fortsatt manuelt, fra fs eller fra web eller SO</li> 
   </ol> 
   

2. select word "om"
3. add link to the selection

Result: link is added next to selection.

Expected Result: Selection should be changed into link.

[Jakub Ś]

#8567 was marked as duplicate.

[Jakub Ś]

TC from #8567

1. Select some text that is in a bullet/numbered list
2. Click on hyperlink icon
3. Choose file from server (can be done on CKEditor demo where CKFinder is attached).
4. URL should point to that file. Click OK.

[Wiktor Walc]

Creating links is a very basic and common operation, so let's fix this bug sooner than later.

[Jakub Ś]

#8719 was marked as duplicate

@josephrider has provided patch which I'm attaching to this ticket.
I did a quick test and this patch has solved this particular problem.

[Jakub Ś]

patch proposed by @josephrider

[Jakub Ś]

Other tickets caused by rev [7243]: #8455, #8475, #8632

[Garry Yao]

Supposed to be fixed by #8345, it shouldn't be related to [7243], pls check.

[Jakub Ś]

I have checked it one more time (followed exactly TC steps). Reproducible from [7243] and [7384] didn't fix it.

checked issue under [7394] but it still persists.

[Garry Yao]

Supposed to be fixed by #8632.

[Jakub Ś]

Seems that all three TC's mentioned here pass. Works as expected

[Garry Yao]

Fixed by #8632.