From: Matías Aguirre <matiasaguirre@gmail.com>
Date: Mon, 17 Jan 2011 21:27:28 +0000 (-0200)
Subject: Simplify extra_data update process
X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=62856a2d980b80e879952ae51d083fd05d568d12;p=django-social-auth.git

Simplify extra_data update process
---

diff --git a/social_auth/backends.py b/social_auth/backends.py
index 0ff6e23..30d780a 100644
--- a/social_auth/backends.py
+++ b/social_auth/backends.py
@@ -61,15 +61,14 @@ class SocialAuthBackend(ModelBackend):
 
         # Update the user account data.
         self.update_user_details(user, response, details, new_user=new_user)
-        
-        # Update the extra_data storage, unless disabled by setting.
-        extra_data = '' if not getattr(settings, 'SOCIAL_AUTH_EXTRA_DATA',
-                                       True) \
-                        else self.extra_data(user, uid, response, details)
-        if extra_data:
-            social_user.extra_data = extra_data
-            social_user.save()
-        
+
+        # Update extra_data storage, unless disabled by setting
+        if getattr(settings, 'SOCIAL_AUTH_EXTRA_DATA', True):
+            extra_data = self.extra_data(user, uid, response, details)
+            if extra_data:
+                social_user.extra_data = extra_data
+                social_user.save()
+
         return user
 
     def username(self, details):