Ensure that artifacts have a timestamp before giving them the timelin…

…e_timestamp property
redhat-exd-rebuilds · Jun 24, 2019 · 389cb90 · 389cb90
1 parent 2af8bd7
commit 389cb90
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 5 deletions.
diff --git a/estuary/models/bugzilla.py b/estuary/models/bugzilla.py
@@ -45,7 +45,9 @@ def display_name(self):
     @property
     def timeline_timestamp(self):
         """Get the DateTime property used for the Estuary timeline."""
-        return self.creation_time.strftime("%Y-%m-%dT%H:%M:%SZ")
+        if self.creation_time:
+            return self.creation_time.strftime('%Y-%m-%dT%H:%M:%SZ')
+        return None
 
     @classmethod
     def find_or_none(cls, identifier):

diff --git a/estuary/models/distgit.py b/estuary/models/distgit.py
@@ -67,4 +67,6 @@ def display_name(self):
     @property
     def timeline_timestamp(self):
         """Get the DateTime property used for the Estuary timeline."""
-        return self.commit_date.strftime("%Y-%m-%dT%H:%M:%SZ")
+        if self.commit_date:
+            return self.commit_date.strftime('%Y-%m-%dT%H:%M:%SZ')
+        return None
diff --git a/estuary/models/errata.py b/estuary/models/errata.py
@@ -44,7 +44,9 @@ def display_name(self):
     @property
     def timeline_timestamp(self):
         """Get the DateTime property used for the Estuary timeline."""
-        return self.created_at.strftime("%Y-%m-%dT%H:%M:%SZ")
+        if self.created_at:
+            return self.created_at.strftime('%Y-%m-%dT%H:%M:%SZ')
+        return None
 
     @classmethod
     def find_or_none(cls, identifier):

diff --git a/estuary/models/freshmaker.py b/estuary/models/freshmaker.py
@@ -33,7 +33,9 @@ def display_name(self):
     @property
     def timeline_timestamp(self):
         """Get the DateTime property used for the Estuary timeline."""
-        return self.time_created.strftime("%Y-%m-%dT%H:%M:%SZ")
+        if self.time_created:
+            return self.time_created.strftime('%Y-%m-%dT%H:%M:%SZ')
+        return None
 
 
 class FreshmakerBuild(EstuaryStructuredNode):

diff --git a/estuary/models/koji.py b/estuary/models/koji.py
@@ -39,7 +39,9 @@ def display_name(self):
     @property
     def timeline_timestamp(self):
         """Get the DateTime property used for the Estuary timeline."""
-        return self.creation_time.strftime("%Y-%m-%dT%H:%M:%SZ")
+        if self.creation_time:
+            return self.creation_time.strftime('%Y-%m-%dT%H:%M:%SZ')
+        return None
 
     @classmethod
     def find_or_none(cls, identifier):