From 290f355b4730fac536d26d565c22461f13f77d48 Mon Sep 17 00:00:00 2001
From: j <0x006A@0x2620.org>
Date: Fri, 28 Oct 2011 20:27:54 +0200
Subject: [PATCH] try to reduce save calls

---
 pandora/item/models.py | 6 +-----
 pandora/item/tasks.py  | 2 +-
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/pandora/item/models.py b/pandora/item/models.py
index dc8ace5d..ec54b5f4 100644
--- a/pandora/item/models.py
+++ b/pandora/item/models.py
@@ -218,8 +218,8 @@ class Item(models.Model):
             response = external_data('getData', {'id': self.itemId})
             if response['status']['code'] == 200:
                 self.external_data = response['data']
-                self.save()
                 self.make_poster(True)
+                self.save()
 
     def expand_connections(self):
         c = self.get('connections')
@@ -817,7 +817,6 @@ class Item(models.Model):
                         update = True
                 if update:
                     self.rendered = False
-                    self.save()
                     self.update_timeline()
                 break
 
@@ -943,7 +942,6 @@ class Item(models.Model):
             if url:
                 data = ox.net.readUrl(url)
                 self.poster.save('poster.jpg', ContentFile(data))
-                self.save()
             elif os.path.exists(poster):
                 with open(poster) as f:
                     data = f.read()
@@ -1006,7 +1004,6 @@ class Item(models.Model):
         if frames:
             heat = [ox.image.getImageHeat(f['path']) for f in frames] 
             self.poster_frame = heat.index(max(heat))
-            self.save()
 
     def get_poster_frame_path(self):
         frames = self.poster_frames()
@@ -1034,7 +1031,6 @@ class Item(models.Model):
            cmd += ['-f', frame]
         p = subprocess.Popen(cmd)
         p.wait()
-        self.save()
         icons = os.path.abspath(os.path.join(settings.MEDIA_ROOT, icon))
         icons = glob(icons.replace('.jpg', '*.jpg'))
         for f in filter(lambda p: not p.endswith('/icon.jpg'), icons):
diff --git a/pandora/item/tasks.py b/pandora/item/tasks.py
index ad1ab12a..f1a7dacc 100644
--- a/pandora/item/tasks.py
+++ b/pandora/item/tasks.py
@@ -17,7 +17,7 @@ def update_poster(itemId):
     item = models.Item.objects.get(itemId=itemId)
     item.make_poster(True)
     item.make_icon()
-
+    item.save()
 
 @task(ignore_resulsts=True, queue='default')
 def update_external(itemId):