svn commit: r1179261 - /incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java

[ps...@apache.org]

Author: psharples
Date: Wed Oct  5 15:27:01 2011
New Revision: 1179261

URL: http://svn.apache.org/viewvc?rev=1179261&view=rev
Log:
The missing doctype problem had returned. See WOOKIE-236. It would appear that we have to set the omitdoctypedeclaration(false) call. This now appears to remedy the problem.

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java

Modified: incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java?rev=1179261&r1=1179260&r2=1179261&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java Wed Oct  5 15:27:01 2011
@@ -48,11 +48,11 @@ public class HtmlCleaner implements IHtm
 		// set cleaner properties	
 		properties  = cleaner.getProperties();
 		//
-		// The following setting has been removed as it can cause issues with IE going
-		// into "quirks mode" - see WOOKIE-236
-		//
-		// properties.setOmitDoctypeDeclaration(true);
+		// The following setting has been re-inserted. Set to true as it can cause issues with IE going
+		// into "quirks mode" - see WOOKIE-236. 
+		// However, it appears we explicitly have to declare it as false to work correctly.
 		//
+		properties.setOmitDoctypeDeclaration(false);
 		properties.setOmitXmlDeclaration(true);
 		properties.setUseCdataForScriptAndStyle(true);
 		properties.setUseEmptyElementTags(false);