Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 355143 - Bug with CDATA and marshal to contenthandler
Summary: Bug with CDATA and marshal to contenthandler
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-18 14:31 EDT by Denise Smith CLA
Modified: 2022-06-09 10:21 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Denise Smith CLA 2011-08-18 14:31:35 EDT 
CDATA won't work with marshal to contenthandler method.  We should

1- During marshal see if the contenthandler also implements lexicalhandler and if yes than pass it along (instead of null).

2- Modify SAXDocumentBuilder to implement LexicalHandler



To recreate go to test
package org.eclipse.persistence.testing.jaxb.externalizedmetadata.mappings.directcollection.DirectCollectionMappingTestCases

and delete the empty testObjectToContentHandler method
Comment 1 Denise Smith CLA 2011-08-24 13:33:26 EDT 
Additionally delete the same empty method in 
org.eclipse.persistence.testing.jaxb.externalizedmetadata.mappings.direct.DirectMappingTestCases
Comment 2 Denise Smith CLA 2011-08-25 12:51:08 EDT 
Another test to be updated
org.eclipse.persistence.testing.jaxb.annotations.xmlnullpolicy.XmlNullPolicyTestCases
Comment 3 Denise Smith CLA 2011-09-09 16:12:35 EDT 
Another test with the empty method
org.eclipse.persistence.testing.jaxb.annotations.cdata.XmlCDATATestCases and XmlCDATAOverrideTestCases
Comment 4 Blaise Doughan CLA 2011-09-19 11:49:28 EDT 
Another test with empty method:
org.eclipse.persistence.testing.jaxb.annotations.cdata.UnmappedCDATATestCases
Comment 5 Radek Felcman CLA 2018-06-05 02:22:41 EDT 
Fixed as a part of patch for a bug no 534812.
See https://bugs.eclipse.org/bugs/show_bug.cgi?id=534812
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:15:08 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:21:28 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink