Fixes #4274: Fix for logos in player.

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

diff --git a/src/stats/models.py b/src/stats/models.py
index c1b4d16..ca0b55d 100644 (file)
--- a/src/stats/models.py
+++ b/src/stats/models.py
@@ -5,20 +5,21 @@ from django.conf import settings
 from django.db import models
 
 
-class Visits(models.Model):
+class VisitsBase(models.Model):
     book = models.ForeignKey('catalogue.Book', models.CASCADE)
-    year = models.PositiveSmallIntegerField()
-    month = models.PositiveSmallIntegerField()
+    date = models.DateField()
     views = models.IntegerField()
     unique_views = models.IntegerField()
 
+    class Meta:
+        abstract = True
+
     @classmethod
-    def build_month(cls, year, month):
+    def build_for_date(cls, date, period):
         Book = apps.get_model('catalogue', 'Book')
-        ### TODO: Delete existing?
 
-        date = f'{year}-{month:02d}'
-        url = f'{settings.PIWIK_URL}?date={date}&filter_limit=-1&format=CSV&idSite={settings.PIWIK_SITE_ID}&language=pl&method=Actions.getPageUrls&module=API&period=month&segment=&token_auth={settings.PIWIK_TOKEN}&flat=1'
+        date = date.isoformat()
+        url = f'{settings.PIWIK_URL}?date={date}&filter_limit=-1&format=CSV&idSite={settings.PIWIK_SITE_ID}&language=pl&method=Actions.getPageUrls&module=API&period={period}&segment=&token_auth={settings.PIWIK_TOKEN}&flat=1'
         data = urlopen(url).read().decode('utf-16')
         lines = data.split('\n')[1:]
         for line in lines:
@@ -33,7 +34,20 @@ class Visits(models.Model):
                 except Book.DoesNotExist:
                     continue
                 else:
-                    cls.objects.create(
-                        book=book, year=year, month=month,
-                        views=views, unique_views=uviews
+                    cls.objects.update_or_create(
+                        book=book, date=date,
+                        defaults={'views': views, 'unique_views': uviews}
                     )
+
+
+class Visits(VisitsBase):
+    @classmethod
+    def build_month(cls, date):
+        cls.build_for_date(date.replace(day=1), 'month')
+
+
+class DayVisits(VisitsBase):
+    @classmethod
+    def build_day(cls, date):
+        cls.build_for_date(date, 'day')
+