style fix

[wolnelektury.git] / src / sponsors / models.py

diff --git a/src/sponsors/models.py b/src/sponsors/models.py
index c193f42..2fb4d89 100644 (file)
--- a/src/sponsors/models.py
+++ b/src/sponsors/models.py
@@ -1,16 +1,14 @@
-# This file is part of Wolnelektury, licensed under GNU Affero GPLv3 or later.
-# Copyright © Fundacja Nowoczesna Polska. See NOTICE for more information.
+# This file is part of Wolne Lektury, licensed under GNU Affero GPLv3 or later.
+# Copyright © Fundacja Wolne Lektury. See NOTICE for more information.
 #
 import json
 import time
 from io import BytesIO
 from django.core.cache import cache
 from django.db import models
-from django.utils.translation import ugettext_lazy as _
 from django.template.loader import render_to_string
 from PIL import Image
 
-from jsonfield import JSONField
 from django.core.files.base import ContentFile
 
 THUMB_WIDTH = 120
@@ -18,10 +16,10 @@ THUMB_HEIGHT = 120
 
 
 class Sponsor(models.Model):
-    name = models.CharField(_('name'), max_length=120)
-    _description = models.CharField(_('description'), blank=True, max_length=255)
-    logo = models.ImageField(_('logo'), upload_to='sponsorzy/sponsor/logo')
-    url = models.URLField(_('url'), blank=True)
+    name = models.CharField('nazwa', max_length=120)
+    _description = models.CharField('opis', blank=True, max_length=255)
+    logo = models.ImageField('logo', upload_to='sponsorzy/sponsor/logo')
+    url = models.URLField('url', blank=True)
 
     def __str__(self):
         return self.name
@@ -34,15 +32,18 @@ class Sponsor(models.Model):
 
 
 class SponsorPage(models.Model):
-    name = models.CharField(_('name'), max_length=120)
-    sponsors = JSONField(_('sponsors'), default={})
+    name = models.CharField('nazwa', max_length=120)
+    sponsors = models.TextField('sponsorzy', default='{}')
     _html = models.TextField(blank=True, editable=False)
     sprite = models.ImageField(upload_to='sponsorzy/sprite', blank=True)
 
+    def get_sponsors_json(self):
+        return json.loads(self.sponsors or '[]')
+
     def populated_sponsors(self):
         result = []
         offset = 0
-        for column in self.sponsors:
+        for column in self.get_sponsors_json():
             result_group = {'name': column['name'], 'sponsors': []}
             sponsor_objects = Sponsor.objects.in_bulk(column['sponsors'])
             for sponsor_pk in column['sponsors']:
@@ -51,12 +52,13 @@ class SponsorPage(models.Model):
                     offset -= THUMB_HEIGHT
                 except KeyError:
                     pass
-            result.append(result_group)
+            if result_group['sponsors']:
+                result.append(result_group)
         return result
 
     def render_sprite(self):
         sponsor_ids = []
-        for column in self.sponsors:
+        for column in self.get_sponsors_json():
             sponsor_ids.extend(column['sponsors'])
         sponsors = Sponsor.objects.in_bulk(sponsor_ids)
         sprite = Image.new('RGBA', (THUMB_WIDTH, len(sponsors) * THUMB_HEIGHT))
@@ -87,16 +89,13 @@ class SponsorPage(models.Model):
     html = property(fget=html)
 
     def save(self, *args, **kwargs):
-        if isinstance(self.sponsors, str):
-            # Walkaround for weird jsonfield 'no-decode' optimization.
-            self.sponsors = json.loads(self.sponsors)
         self.render_sprite()
         self._html = render_to_string('sponsors/page.html', {
             'sponsors': self.populated_sponsors(),
             'page': self
         })
         ret = super(SponsorPage, self).save(*args, **kwargs)
-        cache.delete('sponsor_page:' + name)
+        cache.delete('sponsor_page:' + self.name)
         return ret
 
     def __str__(self):