use TempFileCleanup class to cleanup poster files after use

[packages/griffith.git] / Kodi.py

diff --git a/Kodi.py b/Kodi.py
index 87dea0a6bd4bba40990081639ac2c5d6252ce3ab..98f1566f92d3f34a50bd97dc92cdf18d150cd65a 100644 (file)
--- a/Kodi.py
+++ b/Kodi.py
@@ -24,14 +24,12 @@ __revision__ = '$Id: $'
 from plugins.imp import ImportPlugin as IP
 import os
 import gutils
-import string
-from xml.dom import minidom, Node
 from lxml import etree
-import zipfile
-from shutil import rmtree
-from tempfile import mkdtemp
-
+import tempfile
+from urllib2 import urlopen, URLError, HTTPError
+from movie import TempFileCleanup
 import logging
+
 log = logging.getLogger("Griffith")
 
 """
@@ -54,6 +52,7 @@ class ImportPlugin(IP):
     xml          = None
     items        = None
     itemindex    = 0
+    poster_file  = None
 
     # used by get_movie_details method
     # griffith field => kodi field
@@ -90,6 +89,8 @@ class ImportPlugin(IP):
         if not IP.initialize(self):
             return False
         self.edit = False
+        self.tempfiles = TempFileCleanup()
+
         return True
 
     def set_source(self, name):
@@ -109,9 +110,13 @@ class ImportPlugin(IP):
             self.fileversion = None
             self.items = None
             self.itemindex = 0
+            if self.poster_file:
+                os.unlink(self.poster_file)
+                self.poster_file = None
 
     def destroy(self):
         """close all resources"""
+        self.tempfiles = None
         IP.destroy(self)
 
     def read_fileversion(self):
@@ -194,7 +199,39 @@ class ImportPlugin(IP):
         if notes:
             details['notes'] = "\n".join(notes)
 
+        # handle poster
+        # take first <thumb aspect="poster"> element
+        posters = item.xpath('thumb[@aspect="poster"]')
+        if posters:
+            self.poster_file = self.grab_url(posters[0].get('preview'), prefix = 'poster_', suffix = '.jpg')
+            details['image'] = self.poster_file
+
         # increment for next iteration
         self.itemindex = self.itemindex + 1
 
         return details
+
+    # grab url, return temp filename with remote file contents
+    # XXX could not figure out how to use griffith own downloader with ui interaction, etc
+    # XXX: grabbing urls while processing import xml blocks the ui
+    def grab_url(self, url, prefix = None, suffix=None):
+        (fd, local_file) = tempfile.mkstemp(suffix=suffix, prefix=prefix)
+        log.debug("Downloading: %s as %s" % (url, local_file))
+        try:
+            f = urlopen(url)
+            os.write(fd, f.read())
+            os.close(fd)
+
+        except HTTPError, e:
+            log.error("HTTP Error: %s: %s" % (e.code, url))
+            return None
+        except URLError, e:
+            log.error("URL Error: %s: %s" % (e.reason, url))
+            return None
+        else:
+            self.tempfiles._tempfiles.append(local_file)
+            return local_file
+
+        # we get here with an exception, cleanup and return None
+        os.unlink(local_file)
+        return None