Opened 12 years ago

Closed 12 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 12 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Alfonso Martínez de Lizarrondo

Attachment: 9219.patch added

Proposed patch

comment:1 Changed 12 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 12 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:3 Changed 12 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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy