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

Bug 338111

Summary: Separate the HTML validator into the org.eclipse.wst.html.core bundle so it can be used in headless tools
Product: [WebTools] WTP Source Editing Reporter: Troy Bishop <tjbishop>
Component: wst.htmlAssignee: Troy Bishop <tjbishop>
Status: RESOLVED FIXED QA Contact: Nick Sandonato <nsand.dev>
Severity: enhancement    
Priority: P3 CC: thatnitind
Version: 3.2.2Keywords: plan
Target Milestone: 3.4 M4Flags: nsand.dev: review+
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
possible patch nsand.dev: iplog+

Description Troy Bishop CLA 2011-02-24 11:08:55 EST 
Build Identifier: WTP 3.2

The HTML validator currently resides in the org.eclipse.wst.html.ui bundle which has UI dependencies and therefore cannot be packaged in headless tools.  Attached is a patch which separates the HTML validator code to org.eclipse.wst.html.core and also ensures that the "as-you-type" validation stays in the org.eclipse.wst.html.ui bundle.

Reproducible: Always
Comment 1 Troy Bishop CLA 2011-02-24 11:09:14 EST 
Created attachment 189708 [details]
possible patch
Comment 2 Nick Sandonato CLA 2011-03-07 10:37:22 EST 
Thanks for the patch, Troy. I'll take a look.
Comment 3 Nick Sandonato CLA 2011-05-12 15:53:02 EDT 
*** Bug 305914 has been marked as a duplicate of this bug. ***
Comment 4 Nick Sandonato CLA 2011-11-09 16:39:52 EST 
Patch looks good to me. Thanks, Troy! I've committed the changes to HEAD.
Comment 5 Juris Sudmalis CLA 2012-07-17 04:46:34 EDT 
See Bug 385195 for a bug possibly caused by this change.
Apparently sometimes just reading the patch isn't enough.

P.S. keep the discussion in that bug topic, not here.
Comment 6 Nitin Dahyabhai CLA 2012-07-17 12:39:32 EDT 
They why comment here?  The validator class itself has never been API, nor will it ever be API.