Opened 9 years ago
Closed 9 years ago
#13003 closed Bug (fixed)
Anchors are uploaded
| Reported by: | Piotr Jasiun | Owned by: | Artur Delura |
|---|---|---|---|
| Priority: | Must have (possibly next milestone) | Milestone: | CKEditor 4.5.0 |
| Component: | General | Version: | 4.5.0 Beta |
| Keywords: | Cc: |
Description
- Open http://ckeditor.dev/plugins/uploadwidget/dev/upload.html
- Drag anchor and drop it somewhere else in the editor.
Result: Anchor is uploaded.
Change History (9)
comment:1 Changed 9 years ago by
| Milestone: | CKEditor 4.5.0 Beta → CKEditor 4.5.0 |
|---|
comment:2 Changed 9 years ago by
| Status: | new → confirmed |
|---|
comment:3 Changed 9 years ago by
| Priority: | Normal → High |
|---|
comment:4 Changed 9 years ago by
| Version: | → 4.5.0 Beta |
|---|
comment:5 Changed 9 years ago by
| Owner: | set to Artur Delura |
|---|---|
| Status: | confirmed → assigned |
comment:6 Changed 9 years ago by
comment:7 Changed 9 years ago by
Done by checking whether element if a fake object. Changes and tests in branch:t/13003.
comment:8 Changed 9 years ago by
| Status: | assigned → review |
|---|
comment:9 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review → closed |
Fixed on major with git:26b9d35.
Note: See
TracTickets for help on using
tickets.
We can prevent this by not calling paste event callback here by adding such check:
if ( imgs.getItem( 0 ) && imgs.getItem( 0 ).hasClass( 'cke_anchor' ) ) { return; }or at the very beginning by using some simple regular expression.
The question is whether we have more cases like this one. And whether we should handle it using some more general approach.