Merge remote-tracking branch 'origin/master'

[redakcja.git] / apps / catalogue / models / book.py

diff --git a/apps/catalogue/models/book.py b/apps/catalogue/models/book.py
index 7c2bccc..a426b08 100755 (executable)
--- a/apps/catalogue/models/book.py
+++ b/apps/catalogue/models/book.py
@@ -8,7 +8,7 @@ from django.db import models, transaction
 from django.template.loader import render_to_string
 from django.utils.translation import ugettext_lazy as _
 from django.conf import settings
-from slughifi import slughifi
+from slugify import slugify
 
 
 import apiclient
@@ -87,6 +87,12 @@ class Book(models.Model):
             self.get_absolute_url()
         )
 
+    def gallery_path(self):
+        return os.path.join(settings.MEDIA_ROOT, settings.IMAGE_DIR, self.gallery)
+
+    def gallery_url(self):
+        return '%s%s%s/' % (settings.MEDIA_URL, settings.IMAGE_DIR, self.gallery)
+
     # Creating & manipulating
     # =======================
 
@@ -94,7 +100,7 @@ class Book(models.Model):
         return self.public or request.user.is_authenticated()
 
     @classmethod
-    @transaction.commit_on_success
+    @transaction.atomic
     def create(cls, creator, text, *args, **kwargs):
         b = cls.objects.create(*args, **kwargs)
         b.chunk_set.all().update(creator=creator)
@@ -106,7 +112,7 @@ class Book(models.Model):
         return self.chunk_set.reverse()[0].split(*args, **kwargs)
 
     @classmethod
-    @transaction.commit_on_success
+    @transaction.atomic
     def import_xml_text(cls, text=u'', previous_book=None,
                 commit_args=None, **kwargs):
         """Imports a book from XML, splitting it into chunks as necessary."""
@@ -127,7 +133,7 @@ class Book(models.Model):
             if not title:
                 title = u'część %d' % (i + 1)
 
-            slug = slughifi(title)
+            slug = slugify(title)
 
             if i < book_len:
                 chunk = instance[i]
@@ -153,7 +159,7 @@ class Book(models.Model):
             i += 1
         return new_slug
 
-    @transaction.commit_on_success
+    @transaction.atomic
     def append(self, other, slugs=None, titles=None):
         """Add all chunks of another book to self."""
         assert self != other
@@ -167,7 +173,7 @@ class Book(models.Model):
         if titles is not None:
             assert len(titles) == len_other
             if slugs is None:
-                slugs = [slughifi(t) for t in titles]
+                slugs = [slugify(t) for t in titles]
 
         for i, chunk in enumerate(other):
             # move chunk to new book
@@ -213,14 +219,14 @@ class Book(models.Model):
         other.delete()
 
 
-    @transaction.commit_on_success
+    @transaction.atomic
     def prepend_history(self, other):
         """Prepend history from all the other book's chunks to own."""
         assert self != other
 
         for i in range(len(self), len(other)):
             title = u"pusta część %d" % i
-            chunk = self.add(slughifi(title), title)
+            chunk = self.add(slugify(title), title)
             chunk.commit('')
 
         for i in range(len(other)):
@@ -416,14 +422,18 @@ class Book(models.Model):
                 parse_dublincore=parse_dublincore,
                 strict=strict)
 
-    def publish(self, user):
+    def publish(self, user, fake=False, host=None):
         """
             Publishes a book on behalf of a (local) user.
         """
         self.assert_publishable()
         changes = self.get_current_changes(publishable=True)
         book_xml = self.materialize(changes=changes)
-        apiclient.api_call(user, "books/", {"book_xml": book_xml})
+        if not fake:
+            data = {"book_xml": book_xml}
+            if host:
+                data['gallery_url'] = host + self.gallery_url()
+            apiclient.api_call(user, "books/", data)
         # record the publish
         br = BookPublishRecord.objects.create(book=self, user=user)
         for c in changes: