(fix) make sure we don't double-append domain part (fixes #3614)

Alinto · May 6, 2016 · a26b2ec · a26b2ec
1 parent fbd2f15
commit a26b2ec
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/SoObjects/SOGo/SOGoUserManager.m b/SoObjects/SOGo/SOGoUserManager.m
@@ -391,12 +391,16 @@ - (NSString *) getExternalLoginForUID: (NSString *) uid
       if ([dd forceExternalLoginWithEmail])
         {
           sd = [SOGoSystemDefaults sharedSystemDefaults];
-          if ([sd enableDomainBasedUID])
-            // On multidomain environment we must use uid@domain
-            // for getEmailForUID method
-            login = [NSString stringWithFormat: @"%@@%@", uid, domain];
+          if ([sd enableDomainBasedUID] &&
+              [login rangeOfString: @"@"].location == NSNotFound)
+            {
+              // On multidomain environment we must use uid@domain
+              // for getEmailForUID method
+              login = [NSString stringWithFormat: @"%@@%@", uid, domain];
+            }
           else
             login = uid;
+
           login = [self getEmailForUID: login];
         }
       else