From: Senthil Kumaran Date: Tue, 28 Dec 2010 16:10:56 +0000 (+0000) Subject: Merged revisions 87542 via svnmerge from X-Git-Tag: v3.2.1b1~347^2~127 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6c85838489d4627d0f8292c3a3aead3519d4765c;p=python Merged revisions 87542 via svnmerge from svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r87542 | senthil.kumaran | 2010-12-28 23:55:16 +0800 (Tue, 28 Dec 2010) | 3 lines Fix Issue10759 - html.parser.unescape() fails on HTML entities with incorrect syntax ........ --- diff --git a/Lib/html/parser.py b/Lib/html/parser.py index c2c7f6bf5d..3f68e189c4 100644 --- a/Lib/html/parser.py +++ b/Lib/html/parser.py @@ -367,13 +367,16 @@ class HTMLParser(_markupbase.ParserBase): return s def replaceEntities(s): s = s.groups()[0] - if s[0] == "#": - s = s[1:] - if s[0] in ['x','X']: - c = int(s[1:], 16) - else: - c = int(s) - return chr(c) + try: + if s[0] == "#": + s = s[1:] + if s[0] in ['x','X']: + c = int(s[1:], 16) + else: + c = int(s) + return chr(c) + except ValueError: + return '&#'+ s +';' else: # Cannot use name2codepoint directly, because HTMLParser # supports apos, which is not part of HTML 4 diff --git a/Lib/test/test_htmlparser.py b/Lib/test/test_htmlparser.py index e982218dba..661d41dce9 100755 --- a/Lib/test/test_htmlparser.py +++ b/Lib/test/test_htmlparser.py @@ -319,6 +319,10 @@ DOCTYPE html [ self._run_check("", [ ("starttag", "html", [("foo", "\u20AC&aa&unsupported;")]) ]) + def test_unescape_function(self): + p = html.parser.HTMLParser() + self.assertEqual(p.unescape('&#bad;'),'&#bad;') + self.assertEqual(p.unescape('&'),'&') def test_main():