Guard against 'view' that is set to None

Crunch-io · Feb 2, 2018 · 70a1275 · 70a1275
1 parent b8f3261
commit 70a1275
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/cr/cube/dimension.py b/src/cr/cube/dimension.py
@@ -16,10 +16,13 @@ def __init__(self, dim, selections=None):
 
     @property
     def subtotals(self):
-        insertions_data = self._dim.get('references', {}) \
-                .get('view', {}) \
-                .get('transform', {}) \
-                .get('insertions', [])
+        view = self._dim.get('references', {}).get('view', {})
+
+        if not view:
+            # View can be both None and {}, thus the edge case.
+            return []
+
+        insertions_data = view.get('transform', {}).get('insertions', [])
         subtotals = [Subtotal(data) for data in insertions_data]
         return [subtotal for subtotal in subtotals if subtotal.is_valid]