From 7d98980121531c8db21256e85590007f621c331c Mon Sep 17 00:00:00 2001
From: Philip Sargent <philip.sargent@gmail.com>
Date: Sun, 29 Jan 2023 22:11:00 +0000
Subject: Fixing dates on expedition table

---
 core/models/troggle.py | 58 +++++++++++++++++++++++++-------------------------
 1 file changed, 29 insertions(+), 29 deletions(-)

(limited to 'core/models/troggle.py')

diff --git a/core/models/troggle.py b/core/models/troggle.py
index 5ef95c8..4651955 100644
--- a/core/models/troggle.py
+++ b/core/models/troggle.py
@@ -82,33 +82,33 @@ class Expedition(TroggleModel):
         return urljoin(settings.URL_ROOT, reverse('expedition', args=[self.year]))
     
     # construction function.  should be moved out
-    def get_expedition_day(self, date):
-        expeditiondays = self.expeditionday_set.filter(date=date)
-        if expeditiondays:
-            if len(expeditiondays) == 1:
-                return expeditiondays[0]
-            else:
-                message =f'! - More than one expeditionday for the same date: {date} .\n  - This should never happen. \n  - Restart mysql and run reset to clean database.'
-                DataIssue.objects.create(parser='expedition', message=message)
-                return expeditiondays[0]
-        res = ExpeditionDay(expedition=self, date=date)
-        res.save()
-        return res
+    # def get_expedition_day(self, date):
+        # expeditiondays = self.expeditionday_set.filter(date=date)
+        # if expeditiondays:
+            # if len(expeditiondays) == 1:
+                # return expeditiondays[0]
+            # else:
+                # message =f'! - More than one expeditionday for the same date: {date} .\n  - This should never happen. \n  - Restart mysql and run reset to clean database.'
+                # DataIssue.objects.create(parser='expedition', message=message)
+                # return expeditiondays[0]
+        # res = ExpeditionDay(expedition=self, date=date)
+        # res.save()
+        # return res
         
-    def day_min(self):
-        """First day of expedition
-        """
-        res = self.expeditionday_set.all()
-        return res and res[0] or None
+    # def day_min(self):
+        # """First day of expedition
+        # """
+        # res = self.expeditionday_set.all()
+        # return res and res[0] or None
     
-    def day_max(self):
-        """last day of expedition
-        """
-        res = self.expeditionday_set.all()
-        return res and res[len(res) - 1] or None
+    # def day_max(self):
+        # """last day of expedition
+        # """
+        # res = self.expeditionday_set.all()
+        # return res and res[len(res) - 1] or None
 
 class ExpeditionDay(TroggleModel):
-    """Exists only so that we can get all logbook trips on this day
+    """Exists only on Expedition now. Removed from logbookentry, persontrip, survex stuff etc.
     """
     expedition  = models.ForeignKey("Expedition",on_delete=models.CASCADE)
     date        = models.DateField()
@@ -209,10 +209,10 @@ class PersonExpedition(TroggleModel):
         return sum([survexblock.legslength  for survexblock in set(survexblocks)])
     
     # would prefer to return actual person trips so we could link to first and last ones
-    def day_min(self):
-        res = self.persontrip_set.aggregate(day_min=Min("expeditionday__date"))
-        return res["day_min"]
+    # def day_min(self):
+        # res = self.persontrip_set.aggregate(day_min=Min("expeditionday__date"))
+        # return res["day_min"]
 
-    def day_max(self):
-        res = self.persontrip_set.all().aggregate(day_max=models.Max("expeditionday__date"))
-        return res["day_max"]
+    # def day_max(self):
+        # res = self.persontrip_set.all().aggregate(day_max=models.Max("expeditionday__date"))
+        # return res["day_max"]
-- 
cgit v1.2.3