Added result caching for the Person.email() method.

- Legacy-Id: 14797
2018-03-14 19:14:16 +00:00 · 2018-03-14 19:14:16 +00:00 · 98f64dfea4
parent f7130529d6
commit 98f64dfea4
1 changed files with 6 additions and 4 deletions
--- a/ietf/person/models.py
+++ b/ietf/person/models.py
@ -104,10 +104,12 @@ class PersonInfo(models.Model):
            return e[0]
        return None
    def email(self):
-        e = self.email_set.filter(primary=True).first()
-        if not e:
-            e = self.email_set.filter(active=True).order_by("-time").first()
-        return e
+        if not hasattr(self, '_cached_email'):
+            e = self.email_set.filter(primary=True).first()
+            if not e:
+                e = self.email_set.filter(active=True).order_by("-time").first()
+            self._cached_email = e
+        return self._cached_email
    def email_address(self):
        e = self.email()
        if e: