diff options
author | Philip Sargent <philip.sargent@gmail.com> | 2023-01-29 22:11:00 +0000 |
---|---|---|
committer | Philip Sargent <philip.sargent@gmail.com> | 2023-01-29 22:11:00 +0000 |
commit | 7d98980121531c8db21256e85590007f621c331c (patch) | |
tree | c2f732ac7da6ee2f26bb2b02b06810806e5245db /core/models/troggle.py | |
parent | 89d0e1723ec57c5472eb1073fc1af5bb587bb7dc (diff) | |
download | troggle-7d98980121531c8db21256e85590007f621c331c.tar.gz troggle-7d98980121531c8db21256e85590007f621c331c.tar.bz2 troggle-7d98980121531c8db21256e85590007f621c331c.zip |
Fixing dates on expedition table
Diffstat (limited to 'core/models/troggle.py')
-rw-r--r-- | core/models/troggle.py | 58 |
1 files changed, 29 insertions, 29 deletions
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"] |