Opened 10 years ago

Closed 10 years ago

#3612 closed Bug (fixed)

Testcase for parsing XML with   in it

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone:
Component: QA Version: SVN (CKEditor) - OLD
Keywords: Review+ Cc:

Description

According to the approved patch in #3577, the XML parser should work even if the data contains  

This adds two testcases to verify that the XMl parser works as expected, and so a new patch for xml.js is needed to add the same feature for non-ie browsers.

Attachments (3)

3612.patch (1.2 KB) - added by Alfonso Martínez de Lizarrondo 10 years ago.
Updated testcase
3612_1.patch (549 bytes) - added by Alfonso Martínez de Lizarrondo 10 years ago.
Patch for non-IE browsers
3612_2.patch (859 bytes) - added by Alfonso Martínez de Lizarrondo 10 years ago.
Proposed patch

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Alfonso Martínez de Lizarrondo

Attachment: 3612.patch added

Updated testcase

Changed 10 years ago by Alfonso Martínez de Lizarrondo

Attachment: 3612_1.patch added

Patch for non-IE browsers

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

Keywords: Review? added

Changed 10 years ago by Alfonso Martínez de Lizarrondo

Attachment: 3612_2.patch added

Proposed patch

comment:2 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Sorry, obviously 3612_1 is far from optimal, the good patches are: 3612 as testcase and 3612_2 as fix for the problem.

comment:3 Changed 10 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:4 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: newclosed

Fixed with [3561]

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