[Patches] [ python-Patches-669683 ] HTMLParser -- allow comma in unquoted attribute values

SourceForge.net noreply@sourceforge.net
Fri, 14 Mar 2003 08:35:21 -0800


Patches item #669683, was opened at 2003-01-17 06:15
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=669683&group_id=5470

Category: Library (Lib)
Group: Python 2.3
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: j paulson (fantoozler)
Assigned to: Fred L. Drake, Jr. (fdrake)
>Summary: HTMLParser -- allow comma in unquoted attribute values

Initial Comment:
An HTML document in the wild had the tag:

<font size="-1" color=rgb(175,18,3)>

and HTMLParser was choking on the "," after the 
"175".  By adding "," to the list of allowed characters 
in attribute values, HTMLParser accepts the document.


----------------------------------------------------------------------

>Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-03-14 11:35

Message:
Logged In: YES 
user_id=3066

The regression test does not appear to have been attached; I
see four copies of essentially the same patch.

That's OK though; I have enough to go on.  I've incorporated
the patch in both Lib/HTMLParser.py 1.12 and Lib/sgmllib.py
1.42 (so it also fixes htmllib).  Regression tests have been
added to Lib/test/test_htmlparser.py 1.10 and
Lib/test/test_sgmllib.py 1.5.

----------------------------------------------------------------------

Comment By: j paulson (fantoozler)
Date: 2003-01-24 19:18

Message:
Logged In: YES 
user_id=690612

Added test case to Lib/test/test_htmlparser.py in addition to the 
HTMLParser.py patch

----------------------------------------------------------------------

Comment By: j paulson (fantoozler)
Date: 2003-01-17 13:46

Message:
Logged In: YES 
user_id=690612

I'll attach the patch file again.  

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2003-01-17 09:17

Message:
Logged In: YES 
user_id=33168

Was this supposed to be a patch or a bug report?  There is
no patch attached.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=669683&group_id=5470