Return User to avoid garbage user creation. Closes gh-169

diff --git a/social_auth/backends/pipeline/social.py b/social_auth/backends/pipeline/social.py
index b1ad80ce087a1c615235202fd461b876f1f28775..6b241acc932e58a049f77ed3550239c329906818 100644 (file)
--- a/social_auth/backends/pipeline/social.py
+++ b/social_auth/backends/pipeline/social.py
@@ -18,9 +18,12 @@ def social_auth_user(backend, uid, user=None, *args, **kwargs):
     except UserSocialAuth.DoesNotExist:
         social_user = None
 
-    if user and social_user and social_user.user != user:
-        raise ValueError('Account already in use.', social_user)
-    return {'social_user': social_user}
+    if social_user:
+        if user and social_user.user != user:
+            raise ValueError('Account already in use.', social_user)
+        elif not user:
+            user = social_user.user
+    return {'social_user': social_user, 'user': user}
 
 
 def associate_user(backend, user, uid, social_user=None, *args, **kwargs):
@@ -38,7 +41,7 @@ def associate_user(backend, user, uid, social_user=None, *args, **kwargs):
         return social_auth_user(backend, uid, user, social_user=social_user,
                                 *args, **kwargs)
     else:
-        return {'social_user': social}
+        return {'social_user': social, 'user': social.user}
 
 
 def load_extra_data(backend, details, response, social_user, uid, user,