Menu
▾
▴
[Jtidy-devel] [ jtidy-Bugs-2891882 ] Incorrect parsing of <td> attributes
|
From: SourceForge.net <no...@so...> - 2009-11-25 11:34:01
|
Bugs item #2891882, was opened at 2009-11-04 12:39 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=2891882&group_id=13153 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Tidy functionality Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alex Kainov (alexkainov) Assigned to: Adrian Sandor (aditsu) Summary: Incorrect parsing of <td> attributes Initial Comment: INPUT: <td width="14%" bgcolor="#008000"> TIDY OUTPUT: <td width="14%" bgcolor="#008000"> DESIRED OUTPUT: <td style="width:14%;background-color:#008000;"> Error: The tag: "td" doesn't have an attribute: "width" in currently active versions. The tag: "td" doesn't have an attribute: "bgcolor" in currently active versions. Found in version: r918 ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2009-11-25 11:34 Message: Sure, W3C validator is not JTidy :) I got those 2 errors from W3C validator. It mean's JTidy should process those attributes in order to fit XHTML format. I didn't connect desired translation (attribute to style) to any JTydy option. Should I ? ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2009-11-08 01:39 Message: The W3C validator is not JTidy. Did you get those 2 errors from JTidy or not? Also which JTidy option (that you used) is supposed to replace the width and bgcolor attributes with inline style? ---------------------------------------------------------------------- Comment By: Alex Kainov (alexkainov) Date: 2009-11-06 11:00 Message: If you try W3C validator at http://validator.w3.org/#validate_by_upload+with_options with file after.html (see attachment), you'll get errors connected with "width" and "bgcolor" attributes of <td> tag. Short extract of error report you can find in file "ErrsByValidator.doc" (see attachment). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2009-11-06 10:14 Message: "The code is quite simple" -> I totally disagree. Also note that you should attach a file instead of pasting the input in a comment, because it's too big, and because the formatting and encoding were most likely affected. Anyway, I used your code and your input, and I still did not get the errors you mentioned. It did preserve the width and bgcolor attributes in the td, but I'm not sure why you expect it to change them. ---------------------------------------------------------------------- Comment By: Alex Kainov (alexkainov) Date: 2009-11-06 09:38 Message: OK :) Here is the input: <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"><html><head></head><body text="#000000" bgcolor="#FFFFFF"><table width="100%" border="1"><tr valign="top"><td width="14%"><p><font face="Verdana">cell 1</font></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr><tr valign="top"><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><font face="Verdana">row 2 Column 2</font></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr><tr valign="top"><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><font face="Verdana">??? </font><i><font color="#00E100" face="Verdana">??????? ??????</font></i></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr><tr valign="top"><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><font face="Verdana">???????? (</font><font color="#FF0000" face="Verdana">???????</font><font face="Verdana">)</font></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><b><i><u><font color="#A1009F" face="Verdana">?????????? ?????? ?????? ????????????</font></u></i></b></td><td width="14%" bgcolor="#00FFFF"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#008000"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#0000FF"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr><tr valign="top"><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><font color="#0062E1" face="Verdana">????? ???? </font><b><font color="#0062E1" face="Verdana">??????</font></b></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr><tr valign="top"><td width="14%" bgcolor="#FF0000"><font face="Verdana">ura</font></td><td width="14%" bgcolor="#FF8100"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#FFE118"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#00E100"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#00FFFF"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#2181FF"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td><td width="14%" bgcolor="#C200FF"><img width="1" height="1" src="/icons/ecblank.gif" border="0" alt=""></td></tr></table></body></html> ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2009-11-06 09:33 Message: Hi, I asked for the input, but you didn't provide it, you only explained how you obtained it. You're also using TOOOOOOOOOO many steps to convert the input before passing it to JTidy, but that shouldn't affect the processing of the td tag you showed. You don't need any .NET stuff. You can find Tidy at http://tidy.sourceforge.net/ and http://sourceforge.net/projects/tidy ---------------------------------------------------------------------- Comment By: Alex Kainov (alexkainov) Date: 2009-11-06 08:58 Message: Hi ! Thanks for the answer ! Well, I use URL u.openStream() as input for the parser: URL u = new URL(url + doc.getUniversalID()); BufferedReader in = new BufferedReader( new InputStreamReader( u.openStream(),"UTF-8") ); String s; StringBuffer htmlStr = new StringBuffer(); while( (s = in.readLine()) != null){ htmlStr.append(s); } String htmlString = htmlStr.toString(); The code is quite simple: Tidy tidy= new Tidy(); // obtain a new Tidy instance tidy.setDocType("strict"); tidy.setDropFontTags(true); tidy.setFixBackslash(true); tidy.setFixUri(true); tidy.setJoinClasses(true); tidy.setJoinStyles(true); tidy.setLogicalEmphasis(true); tidy.setQuiet(true); tidy.setQuoteMarks(true); tidy.setShowWarnings(false); tidy.setTidyMark(false); tidy.setXHTML(true); tidy.setInputEncoding("UTF8"); tidy.setOutputEncoding("UTF8"); byte currentXMLBytes[] = htmlString.getBytes("UTF-8"); ByteArrayInputStream byteArrayInputStream = new ByteArrayInputStream(currentXMLBytes); ByteArrayOutputStream byteArrayOutputStream= new ByteArrayOutputStream(); tidy.parse(byteArrayInputStream, byteArrayOutputStream); String sBuffer= byteArrayOutputStream.toString("UTF-8"); Concerning your proposal of using tidy (the C program). I've found only links to the program for .NET. .NETis not installed on my computer. Any idea ? Regards, Alex. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2009-11-06 02:32 Message: I don't get those errors, maybe you haven't included the whole input (especially the doctype). You also haven't provided the code. Anyway, check if tidy (the C program) behaves differently. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=2891882&group_id=13153 |