Link dialog removes onclick attribute

[Henning]

Steps to reproduce

1. go to CKEditor demo ​http://ckeditor.com/demo#standard
2. switch to source code mode
3. remove the existing source
4. enter

   <a href="www.google.de" onclick="alert('onclick');" >my link with onclick</a>
   

5. switch to wysiwyg mode
6. doubleclick the link
7. in the link dialog, click the OK button

Expected result

onclick attribute is preserved without change.

Actual result

onclick attribute is removed

Other details (browser, OS, CKEditor version, installed plugins)

I guess that CKEditor removes the onClick attribute because it might have been generated by CKEditor before via the Target option _<popup window>_. But CKEditors link dialog should remove the onClick only if the target option _<popup window>_ was set when the link dialog was opened.

[Jakub Ś]

Problem can be reproduced from CKEditor 3.0. If you insert the code and then edit the link using Link dialog, the onclick attribute is removed.

From what I have checked only the onclick is removed (tested with ACF disabled).

[kkrzton]

Changes in ​t/13694.

Added handling of user defined onclick attribute. Before it was always removed because of popup onlick handling.