Anchor is duplicated when edited

[AlexanderMur]

Type of report

Bug

Provide detailed reproduction steps (if any)

1. go to https://ckeditor.com/latest/samples/index.html
2. make some bold text
3. make it anchor
4. try to edit anchor - double click
5. press ok

example: https://i.gyazo.com/fa442d930a3b59eec754ca0b5956b8b5.mp4

Every time when you anchor is edited with style like bold underline etc. - this makes a doubled anchor instead of just editing.

Expected result

anchor is edited

Actual result

anchor is doubled

Other details

• Browser: Chrome 80.0.3987.106
• OS: Windows 10 1903
• CKEditor version: 4

[Comandeer]

I can confirm the issue. Seems like internal getSelectedAnchor function assumes that text can be surrounded only by the link itself, not considering more complex situations like:

<a><strong>Text</strong></a>

[AlexanderMur]

How can I fix this without modifying source code?

[deyankush]

@Comandeer any updates on this defect? I am facing the same issue. This was working in v4.7. Since we migrated to v4.13 its stopped working.

[Dumluregn]

Hello @deyankush, thank you for bringing to our attention that it's a regression. It was indeed introduced in 4.13.0, which means we probably try to handle it in the near future.

[vinaydhakre]

@Dumluregn - I am facing the same issue. Could you take this defect on priority so that we can get this fix ASAP?

[Dumluregn]

Unfortunately we've just froze the branches before release, so this issue will have to wait for the next one. However if nothing unexpected happens we should be able to fix this for then.

[vaibhav198989]

@Dumluregn when is the next release expected ?

[deyankush]

Hello @deyankush, thank you for bringing to our attention that it's a regression. It was indeed introduced in 4.13.0, which means we probably try to handle it in the near future.

Is it possible to get a patch on 4.13?

[Dumluregn]

@vaibhav198989 we do not have a date set, probably in a few months.
@deyankush we do not provide backwards patches, but if you have a commercial license you can reach our support team for workaround or help in your own development of this fix.

[deyankush]

@Dumluregn this commit seems to have caused the regression: 7c93a77
@jacekbogdanski could you please confirm?

[jacekbogdanski]

Closed in #4733