From: Marek Stępniowski <marek@stepniowski.com>
Date: Mon, 29 Sep 2008 14:06:50 +0000 (+0200)
Subject: Fixed a bug in saving shelves for each bug (beware of lazy evaluation of querysets!).
X-Git-Url: https://git.mdrn.pl/wolnelektury.git/commitdiff_plain/0242f6b40865679bb86cac65e42fc3889155a0f0?ds=inline;hp=-c

Fixed a bug in saving shelves for each bug (beware of lazy evaluation of querysets!).
---

0242f6b40865679bb86cac65e42fc3889155a0f0
diff --git a/apps/catalogue/models.py b/apps/catalogue/models.py
index 18e7e804d..60d278580 100644
--- a/apps/catalogue/models.py
+++ b/apps/catalogue/models.py
@@ -164,7 +164,7 @@ class Book(models.Model):
             if not overwrite:
                 raise Book.AlreadyExists('Book %s already exists' % book_slug)
             # Save shelves for this book
-            book_shelves = book.tags.filter(category='set')
+            book_shelves = list(book.tags.filter(category='set'))
         
         book.title = book_info.title
         book._short_html = ''
@@ -230,7 +230,7 @@ class Book(models.Model):
                 book_themes += themes
             
             book_themes = set(book_themes)
-            book.tags = list(book.tags) + list(book_themes) + list(book_shelves)
+            book.tags = list(book.tags) + list(book_themes) + book_shelves
         
         book.save()
         return book