Opened 4 years ago

Closed 4 years ago

#9219 closed Bug (fixed)

Add <source> to the list of elements to protect

Reported by: alfonsoml Owned by: alfonsoml
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 alfonsoml 4 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by alfonsoml

Proposed patch

comment:1 Changed 4 years ago by alfonsoml

  • Owner set to alfonsoml
  • Status changed from new to review

Of course, something similar should be done for v4

comment:2 Changed 4 years ago by fredck

  • Status changed from review to review_passed

comment:3 Changed 4 years ago by alfonsoml

  • Milestone set to CKEditor 3.6.5
  • Resolution set to fixed
  • Status changed from review_passed to closed

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 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy