Removed dead code.

[bookmarks_db.git] / Robots / bkmk_rsimple.py

diff --git a/Robots/bkmk_rsimple.py b/Robots/bkmk_rsimple.py
index af549f63969001f87e725b90df87811552512491..1842b4d7ff94e73327f127afa2b71a74c61f7ed8 100644 (file)
--- a/Robots/bkmk_rsimple.py
+++ b/Robots/bkmk_rsimple.py
@@ -83,13 +83,16 @@ from bkmk_objects import Robot
 from parse_html import parse_html
 
 class robot_simple(Robot):
-   def check_url(self, bookmark, url_type, url_rest):
+   def check_url(self, bookmark):
       if not self.tempfname:
          self.tempfname = bookmark.tempfname
 
       try:
          try:
             self.start = int(time.time())
+            bookmark.icon = None
+
+            url_type, url_rest = urllib.splittype(bookmark.href)
             url_host, url_path = urllib.splithost(url_rest)
             url_path, url_tag  = urllib.splittag(url_path)
 
@@ -164,7 +167,7 @@ class robot_simple(Robot):
                      icon = parser.icon
                      if not icon:
                         icon = "/favicon.ico"
-                     icon = urljoin("%s://%s" % (url_type, url_host), icon)
+                     icon = urljoin("%s://%s%s" % (url_type, url_host, url_path), icon)
                      if self.log: self.log("   icon           : %s" % icon)
                      try:
                         fname, headers = urllib.urlretrieve(icon)
@@ -180,7 +183,8 @@ class robot_simple(Robot):
                         icon_file.close()
                         os.remove(fname)
                         content_type = headers["Content-Type"]
-                        bookmark.icon = "data:%s;base64,%s" % (content_type, b64encode(icon))
+                        if content_type.startswith("image/"):
+                           bookmark.icon = "data:%s;base64,%s" % (content_type, b64encode(icon))
 
                except KeyError:
                   pass