time_stats(): use an existing attribute if available

Gauvain Pocentek · Gauvain Pocentek · commit f2223e2397ae · 2018-05-30T20:24:57.000+02:00
A time_stats attribute is returned by GitLab when fetching issues and merge requests (on reasonably recent GitLab versions). Use this info instead of making a new API call if possible. Fixes #510
diff --git a/gitlab/mixins.py b/gitlab/mixins.py
@@ -477,6 +477,11 @@ def time_stats(self, **kwargs):
             GitlabAuthenticationError: If authentication is not correct
             GitlabTimeTrackingError: If the time tracking update cannot be done
         """
+        # Use the existing time_stats attribute if it exist, otherwise make an
+        # API call
+        if 'time_stats' in self.attributes:
+            return self.attributes['time_stats']
+
         path = '%s/%s/time_stats' % (self.manager.path, self.get_id())
         return self.manager.gitlab.http_get(path, **kwargs)
 
