Opened 5 years ago

Closed 5 years ago

#9219 closed Bug (fixed)

Add <source> to the list of elements to protect

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone: CKEditor 3.6.5
Component: Core : Parser Version: 3.0
Keywords: Cc:

Description

Using the HTML5 video plugin, http://alfonsoml.blogspot.com.es/2011/01/html5-video-plugin-for-ckeditor.html add an element but put fake files just to notice this more easily.

Switch to source and back to design, in Firebug it's visible that it has tried to load the src files during the processing because <source> isn't in the list of elements whose attributes must be protected.

As this is a variable inside the function it doesn't seem possible to fix it from outside code.

Attachments (1)

9219.patch (690 bytes) - added by Alfonso Martínez de Lizarrondo 5 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by Alfonso Martínez de Lizarrondo

Attachment: 9219.patch added

Proposed patch

comment:1 Changed 5 years ago by Alfonso Martínez de Lizarrondo

Owner: set to Alfonso Martínez de Lizarrondo
Status: newreview

Of course, something similar should be done for v4

comment:2 Changed 5 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:3 Changed 5 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.6.5
Resolution: fixed
Status: review_passedclosed

Fixed with [7619]

I guess that there's no need to leave this ticket open to apply the fix in v4.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy