From 0db27bba87e4d685f27c1e84dd7becc8ef6b83d1 Mon Sep 17 00:00:00 2001
From: Andrew Williams <andy@tensixtyone.com>
Date: Thu, 21 Apr 2011 08:56:02 +0100
Subject: [PATCH] Now kill all access if set is_active = False

---
 app/sso/tasks.py | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/app/sso/tasks.py b/app/sso/tasks.py
index 843c35e..be5a004 100644
--- a/app/sso/tasks.py
+++ b/app/sso/tasks.py
@@ -67,19 +67,19 @@ def update_user_access(user, **kwargs):
             servacc.active = 0
             servacc.save()
             pass
-
-    # For each of the user's services, check they're in a valid group for it and enable/disable as needed.
-    for servacc in ServiceAccount.objects.filter(user=user):
-        if not (set(user.groups.all()) & set(servacc.service.groups.all())):
-            if servacc.active:
-                servacc.active = 0
-                servacc.save()
-                pass
-        else:
-            if not servacc.active:
-                servacc.active = 1
-                servacc.save()
-                pass
+    else:
+        # For each of the user's services, check they're in a valid group for it and enable/disable as needed.
+        for servacc in ServiceAccount.objects.filter(user=user):
+            if not (set(user.groups.all()) & set(servacc.service.groups.all())):
+                if servacc.active:
+                    servacc.active = 0
+                    servacc.save()
+                    pass
+            else:
+                if not servacc.active:
+                    servacc.active = 1
+                    servacc.save()
+                    pass
 
     update_service_groups.delay(user_id=user.id)