See ChangeLog

Monotone-Parent: 707f11d374470774389d0ad41fc35614be923164
Monotone-Revision: d5f67d6e4d96fb56aedcbaef7a68bdf915d7ad8c

Monotone-Author: flachapelle@inverse.ca
Monotone-Date: 2009-05-05T19:58:33
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,9 @@
+2009-05-05  Francis Lachapelle  <flachapelle@inverse.ca>
+
+	* SoObjects/SOGo/SOGoUser.m ([SOGoUser
+	+userWithLogin:roles:trust:]): don't set the roles if the passed
+	roles are not defined.
+
 2009-05-03  Francis Lachapelle  <flachapelle@inverse.ca>
 
 	* UI/Common/UIxUserRightsEditor.m ([UIxUserRightsEditor

SoObjects/SOGo/LDAPUserManager.m

@@ -377,6 +377,7 @@ static NSLock *lock = nil;
 	  if (!access)
 	    [currentUser setObject: [NSNumber numberWithBool: NO]
 			 forKey: @"MailAccess"];
+	  break;
 	}
     }
 

SoObjects/SOGo/SOGoUser.m

@@ -238,14 +238,15 @@ _timeValue (NSString *key)
   user = [cache userNamed: newLogin];
   if (!user)
     {
-      user = [[self alloc] initWithLogin: newLogin roles: newRoles  trust: b];
+      user = [[self alloc] initWithLogin: newLogin  roles: newRoles  trust: b];
       if (user)
 	{
 	  [user autorelease];
  	  [cache registerUser: user];
 	}
     }
-  [user setPrimaryRoles: newRoles];
+  if (newRoles)
+    [user setPrimaryRoles: newRoles];
 
   return user;
 }

UI/Common/UIxUserRightsEditor.m

@@ -93,7 +93,7 @@ static BOOL sendACLAdvisories = NO;
 - (BOOL) _initRights
 {
   BOOL response;
-  NSString *newUID, *searchUID;
+  NSString *newUID;
   LDAPUserManager *um;
   SOGoObject *clientObject;
   SOGoGroup *group;