Opened 14 years ago
Closed 14 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 14 years ago by
Attachment: | test-insertHtml.patch added |
---|
Changed 14 years ago by
Attachment: | 3075.patch added |
---|
comment:1 Changed 14 years ago by
Keywords: | Review? added |
---|---|
Owner: | set to Garry Yao |
Status: | new → assigned |
comment:2 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 14 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 14 years ago by
Attachment: | 3075_2.patch added |
---|
comment:4 Changed 14 years ago by
Keywords: | Review? added; Review- removed |
---|
Fred you are right, we don't need establish this closure.
comment:5 Changed 14 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:6 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Unit Test Case