Remove else block to make enrollment caching path more clear.

openedx · Sep 12, 2017 · 1fbd581 · 1fbd581
1 parent d040ce6
commit 1fbd581
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/ecommerce/programs/conditions.py b/ecommerce/programs/conditions.py
@@ -84,10 +84,9 @@ def is_satisfied(self, offer, basket):  # pylint: disable=unused-argument
                 user = basket.owner.username
                 try:
                     enrollments = api.enrollment.get(user=user)
+                    cache.set(cache_key, enrollments, settings.ENROLLMENT_API_CACHE_TIMEOUT)
                 except (ConnectionError, SlumberBaseException, Timeout) as exc:
                     logger.error('Failed to retrieve enrollments: %s', str(exc))
-                else:
-                    cache.set(cache_key, enrollments, settings.ENROLLMENT_API_CACHE_TIMEOUT)
 
         for course in program['courses']:
             # If the user is already enrolled in a course, we do not need to check their basket for it