X-Git-Url: https://git.mdrn.pl/redakcja.git/blobdiff_plain/5913c54d19b8f6775633176032161d49f9b2f1aa..3204e4303148302d278036eebcfc8cb105cc97d7:/src/catalogue/models/book.py diff --git a/src/catalogue/models/book.py b/src/catalogue/models/book.py index 1fcd05ac..229b3d0b 100755 --- a/src/catalogue/models/book.py +++ b/src/catalogue/models/book.py @@ -15,7 +15,6 @@ import apiclient from catalogue.helpers import cached_in_field, GalleryMerger from catalogue.models import BookPublishRecord, ChunkPublishRecord, Project from catalogue.signals import post_publish -from catalogue.tasks import refresh_instance, book_content_updated from catalogue.xml_tools import compile_text, split_xml from cover.models import Image import os @@ -36,7 +35,6 @@ class Book(models.Model): parent_number = models.IntegerField(_('parent number'), null=True, blank=True, db_index=True, editable=False) # Cache - _short_html = models.TextField(null=True, blank=True, editable=False) _single = models.NullBooleanField(editable=False, db_index=True) _new_publishable = models.NullBooleanField(editable=False) _published = models.NullBooleanField(editable=False) @@ -68,13 +66,13 @@ class Book(models.Model): def __len__(self): return self.chunk_set.count() - def __nonzero__(self): + def __bool__(self): """ Necessary so that __len__ isn't used for bool evaluation. """ return True - def __unicode__(self): + def __str__(self): return self.title @models.permalink @@ -268,12 +266,12 @@ class Book(models.Model): try: bi = self.wldocument(changes=changes, strict=True).book_info - except ParseError, e: - raise AssertionError(_('Invalid XML') + ': ' + unicode(e)) + except ParseError as e: + raise AssertionError(_('Invalid XML') + ': ' + str(e)) except NoDublinCore: raise AssertionError(_('No Dublin Core found.')) - except ValidationError, e: - raise AssertionError(_('Invalid Dublin Core') + ': ' + unicode(e)) + except ValidationError as e: + raise AssertionError(_('Invalid Dublin Core') + ': ' + str(e)) valid_about = self.correct_about() assert bi.about == valid_about, _("rdf:about is not") + " " + valid_about @@ -281,7 +279,7 @@ class Book(models.Model): def publishable_error(self): try: return self.assert_publishable() - except AssertionError, e: + except AssertionError as e: return e else: return None @@ -326,10 +324,6 @@ class Book(models.Model): return len(self) == 1 single = cached_in_field('_single')(is_single) - @cached_in_field('_short_html') - def short_html(self): - return render_to_string('catalogue/book_list/book.html', {'book': self}) - def book_info(self, publishable=True): try: book_xml = self.materialize(publishable=publishable) @@ -363,25 +357,14 @@ class Book(models.Model): Book.objects.filter(pk=self.pk).update(**update) def touch(self): - # this should only really be done when text or publishable status changes - book_content_updated.delay(self) - update = { "_new_publishable": self.is_new_publishable(), "_published": self.is_published(), "_single": self.is_single(), "_on_track": self.get_on_track(), - "_short_html": None, } Book.objects.filter(pk=self.pk).update(**update) - refresh_instance(self) - - def refresh(self): - """This should be done offline.""" - self.short_html - self.single - self.new_publishable - self.published + self.refresh_dc_cache() # Materializing & publishing # ==========================