Opened 16 years ago
Closed 16 years ago
#3075 closed Bug (fixed)
plugin:wysiwyg broken on insertHtml
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | SVN (FCKeditor) - Retired |
| Keywords: | Confirmed Review+ | Cc: |
Description (last modified by )
After introducing the HTML Parser Filter system, method like protectHtml should be removed, which in this case cause JavaScript error.
Attachments (3)
Change History (9)
Changed 16 years ago by
| Attachment: | test-insertHtml.patch added |
|---|
Changed 16 years ago by
| Attachment: | 3075.patch added |
|---|
comment:1 Changed 16 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:2 Changed 16 years ago by
| Description: | modified (diff) |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The reason for having those functions outside the init scope is to avoid them being recreated for each editor instance. Why changing things that are not related to this ticket? Please leave them in their original place, or give it a good reason why.
Changed 16 years ago by
| Attachment: | 3075_2.patch added |
|---|
comment:4 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Fred you are right, we don't need establish this closure.
comment:5 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:6 Changed 16 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Unit Test Case