fixes

diff --git a/apps/catalogue/fields.py b/apps/catalogue/fields.py
index 510c06d..0488244 100644 (file)
--- a/apps/catalogue/fields.py
+++ b/apps/catalogue/fields.py
@@ -122,9 +122,12 @@ class OverwritingFieldFile(FieldFile):
 
     def save(self, name, content, *args, **kwargs):
         leave = kwargs.pop('leave', None)
 
     def save(self, name, content, *args, **kwargs):
         leave = kwargs.pop('leave', None)

-        if not leave and self and content is not self:
+        # delete if there's a file already and there's a new one coming
+        if not leave and self and (not hasattr(content, 'path') or
+                                   content.path != self.path):

             self.delete(save=False)
             self.delete(save=False)

-        return super(OverwritingFieldFile, self).save(name, content, *args, **kwargs)
+        return super(OverwritingFieldFile, self).save(
+                name, content, *args, **kwargs)

 
 
 class OverwritingFileField(models.FileField):
 
 
 class OverwritingFileField(models.FileField):

diff --git a/apps/catalogue/tests/book_import.py b/apps/catalogue/tests/book_import.py
index c062b32..57dbf27 100644 (file)
--- a/apps/catalogue/tests/book_import.py
+++ b/apps/catalogue/tests/book_import.py
@@ -43,7 +43,6 @@ class BookImportLogicTests(WLTestCase):
         # TODO: this should be filled out probably...
         self.assertEqual(book.wiki_link, '')
         self.assertEqual(book.gazeta_link, '')
         # TODO: this should be filled out probably...
         self.assertEqual(book.wiki_link, '')
         self.assertEqual(book.gazeta_link, '')

-        self.assertEqual(book._short_html, '')

         self.assertEqual(book.description, '')
 
         tags = [ (tag.category, tag.slug) for tag in book.tags ]
         self.assertEqual(book.description, '')
 
         tags = [ (tag.category, tag.slug) for tag in book.tags ]