Opened 17 years ago
Closed 17 years ago
#3303 closed Bug (fixed)
Remove list command is broken again
| Reported by: | Martin Kou | Owned by: | Frederico Caldeira Knabben | 
|---|---|---|---|
| Priority: | Must have (possibly next milestone) | Milestone: | CKEditor 3.0 | 
| Component: | General | Version: | SVN (CKEditor) - OLD | 
| Keywords: | Review+ | Cc: | 
Description
To reproduce:
- Open replacebyclass.html in any supported browser.
 - Put caret on the first paragraph.
 - Press Create/Remove Numbered List button - a list is created as expected.
 - Press Create/Remove Numbered List button again - a JavaScript error appears.
 
The error seems to have to do with recent changes to DOM node custom data logic.
Attachments (1)
Change History (4)
Changed 17 years ago by
| Attachment: | 3303.patch added | 
|---|
comment:1 Changed 17 years ago by
| Keywords: | Review? added | 
|---|---|
| Owner: | set to Frederico Caldeira Knabben | 
| Status: | new → assigned | 
comment:2 Changed 17 years ago by
| Keywords: | Review+ added; Review? removed | 
|---|

The original implementation of the "custom data" related functions where assuming that the expando property was always there, which is not true anymore.