Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 341403

Summary: Remove dependency on WTP DTD parser
Product: z_Archived Reporter: Konstantin Komissarchik <konstantin>
Component: SapphireAssignee: Konstantin Komissarchik <konstantin>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: ling.hao
Version: unspecifiedKeywords: plan
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Konstantin Komissarchik CLA 2011-03-30 13:43:40 EDT
When we originally added support for DTDs it made sense to re-use WTP parser as it sped up our implementation. At this point, however, dependency on WTP at this level of the framework restricts certain scenarios where Sapphire could be used. 

Since DTD is such a simple language, we should just implement our own parser.
Comment 1 Konstantin Komissarchik CLA 2011-03-30 19:53:45 EDT
Done.

Added two unit tests to cover DTD parsing, including entity resolution. Also tried the parser with a variety of complex DTDs that I found.
Comment 2 Konstantin Komissarchik CLA 2011-03-30 19:54:00 EDT
Done.
Comment 3 Ling Hao CLA 2011-04-15 13:39:37 EDT
Verified all is well with coherence override 3.5 dtd xml file.