From 65b1967758cd638c1be92febcc6b889f2eae03bb Mon Sep 17 00:00:00 2001
From: Andrew Williams <andy@tensixtyone.com>
Date: Tue, 4 Jan 2011 17:14:28 +0000
Subject: [PATCH] Check if the groups are there to remove/add

---
 sso/tasks.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sso/tasks.py b/sso/tasks.py
index 0baa67b..c904ac1 100644
--- a/sso/tasks.py
+++ b/sso/tasks.py
@@ -42,10 +42,12 @@ def update_user_access(user):
     addgroups = set(chargroups) - set(set(user.groups.all()) & set(corpgroups))
 
     for g in delgroups:
-        user.groups.remove(g)
+        if g in user.groups.all():
+            user.groups.remove(g)
 
     for g in addgroups:
-        user.groups.add(g)
+        if not g in user.groups.all():
+            user.groups.add(g)
 
     # For users set to not active, delete all accounts
     if not user.is_active: