Opened 10 years ago
Closed 10 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 10 years ago by
| Milestone: | CKEditor 4.5.0 Beta → CKEditor 4.5.0 |
|---|
comment:2 Changed 10 years ago by
| Status: | new → confirmed |
|---|
comment:3 Changed 10 years ago by
| Priority: | Normal → High |
|---|
comment:4 Changed 10 years ago by
| Version: | → 4.5.0 Beta |
|---|
comment:5 Changed 10 years ago by
| Owner: | set to Artur Delura |
|---|---|
| Status: | confirmed → assigned |
comment:6 Changed 10 years ago by
comment:7 Changed 10 years ago by
Done by checking whether element if a fake object. Changes and tests in branch:t/13003.
comment:8 Changed 10 years ago by
| Status: | assigned → review |
|---|
comment:9 Changed 10 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.