Modified cach_helper.py to treat cached empty perm lists as a cache hit

chrisglass · Sep 20, 2012 · cc4a395 · cc4a395
1 parent 1c3c6f6
commit cc4a395
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/rulez/rolez/cache_helper.py b/rulez/rolez/cache_helper.py
@@ -73,8 +73,8 @@ def get_roles(user, obj):
     """
     # get roles for the user, if present:
     roles = cache.get(roles_key(user, obj))
-    if roles:
-        # Cache hit
+    if isinstance(roles, list):
+        # Cache hit (a miss returns NoneType rather than an empty list)
         return roles
     else:
         # we need to recompute roles for this model