Some sites put TITLE in HTML without HEAD.

[bookmarks_db.git] / Robots / parse_html_beautifulsoup.py

diff --git a/Robots/parse_html_beautifulsoup.py b/Robots/parse_html_beautifulsoup.py
index 5f010ad698e894e63d86515fa2a0111ad926da4d..209486c4054f015a179d4469ccaa5adc3f8bbe8f 100644 (file)
--- a/Robots/parse_html_beautifulsoup.py
+++ b/Robots/parse_html_beautifulsoup.py
@@ -55,21 +55,30 @@ def parse_html(filename, charset=None):
       root = BadDeclParser(infile, fromEncoding=charset)
    except TypeError:
       return None
-   infile.close()
+   finally:
+      infile.close()
 
-   _charset = root.originalEncoding
    try:
-      title = root.html.head.title.string.encode(_charset)
+      head = root.html.head
    except AttributeError:
       return None
 
-   meta = root.html.head.find(_find_refresh, recursive=False)
+   if head is None:
+      head = root.html # Some sites put TITLE in HTML without HEAD
+
+   _charset = root.originalEncoding
+   try:
+      title = head.title.string.encode(_charset)
+   except AttributeError:
+      title = '' # HEAD but no TITLE
+
+   meta = head.find(_find_refresh, recursive=False)
    if meta:
       refresh = meta.get("content")
    else:
       refresh = None
 
-   meta = root.html.head.find(_find_icon, recursive=False)
+   meta = head.find(_find_icon, recursive=False)
    if meta:
       icon = meta.get("href")
    else: