From 1e985ebf63e690ab3cf39eb876ef8afb1e17e682 Mon Sep 17 00:00:00 2001
From: j <0x006A@0x2620.org>
Date: Wed, 25 May 2011 21:11:08 +0200
Subject: [PATCH] remove files

---
 pandora/item/models.py | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/pandora/item/models.py b/pandora/item/models.py
index fd75b7ff..9cfe86b4 100644
--- a/pandora/item/models.py
+++ b/pandora/item/models.py
@@ -241,9 +241,12 @@ class Item(models.Model):
             tasks.update_poster.delay(self.itemId)
 
     def delete_files(self):
-        shutil.rmtree(self.path(''))
+        path = os.path.join(settings.MEDIA_ROOT, self.path())
+        if os.path.exists(path):
+            shutil.rmtree(path)
 
     def delete(self, *args, **kwargs):
+        self.streams.all().delete()
         self.delete_files()
         super(Item, self).delete(*args, **kwargs)
 
@@ -251,15 +254,10 @@ class Item(models.Model):
         '''
             move all related tables to other and delete self
         '''
-        #FIXME: stream path is wrong after this, should this be dealt with in save?
-        #       its more that streams have to be generated again after merging
-        for stream in self.streams.all():
-            stream.item = other
-            stream.save()
         for l in self.lists.all():
-            l.items.remove(self)
+            l.remove(self)
             if l.items.filter(id=other.id) == 0:
-                l.items.add(other)
+                l.add(other)
         #FIXME: should this really happen for annotations?
         for a in self.annotations.all():
             a.item = other
@@ -270,6 +268,7 @@ class Item(models.Model):
                 f.save()
         self.delete()
         other.save()
+        #FIXME: update poster, stills and streams after this
 
     def get_poster(self):
         poster = {}