From d9ad8b8616735c8f7300223809426f65cee5f297 Mon Sep 17 00:00:00 2001 From: Henrik Levkowetz Date: Sat, 19 Dec 2020 22:52:07 +0000 Subject: [PATCH] Removed some dead code, simplified TimeSlot *_start_time() and *_end_time() methods, and added TimeSlot.local_date(). - Legacy-Id: 18784 --- ietf/meeting/models.py | 34 ++++++++++++---------------------- 1 file changed, 12 insertions(+), 22 deletions(-) diff --git a/ietf/meeting/models.py b/ietf/meeting/models.py index c527efcc8..77afdb008 100644 --- a/ietf/meeting/models.py +++ b/ietf/meeting/models.py @@ -245,7 +245,7 @@ class Meeting(models.Model): for ts in self.timeslot_set.all(): if ts.location_id is None: continue - ymd = ts.time.date() + ymd = ts.local_date() if ymd not in time_slices: time_slices[ymd] = [] slots[ymd] = [] @@ -474,9 +474,6 @@ class TimeSlot(models.Model): t = self.local_start_time() return "%s-%s" % (t.strftime("%H%M"), (t + self.duration).strftime("%H%M")) - def meeting_date(self): - return self.time.date() - def registration(self): # below implements a object local cache # it tries to find a timeslot of type registration which starts at the same time as this slot @@ -541,28 +538,21 @@ class TimeSlot(models.Model): return self._cached_tz def tzname(self): - if self.tz(): - return self.tz().tzname(self.time.replace(tzinfo=None)) + tz = self.tz() + return tz.tzname(self.time.replace(tzinfo=None)) if tz else None def utc_start_time(self): - if self.tz(): - return self.time.astimezone(pytz.utc) - else: - return None + return self.time.astimezone(pytz.utc) def utc_end_time(self): - if self.tz(): - return self.end_time().astimezone(pytz.utc) - else: - return None + return self.end_time().astimezone(pytz.utc) def local_start_time(self): - if self.tz(): - return self.time.astimezone(self.tz()) - else: - return None + tz = self.tz() + return self.time.astimezone(tz) if tz else None def local_end_time(self): - if self.tz(): - return self.end_time().astimezone(self.tz()) - else: - return None + tz = self.tz() + return self.end_time().astimezone(tz) if tz else None + def local_date(self): + tz = self.tz() + return self.time.astimezone(tz).date() if tz else None @property def js_identifier(self):