Kill that annoying problem that when users without a valid email at register time...

git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@218 0cfe37f9-358a-4d5e-be75-b63607b5c754

diff --git a/forum/views/auth.py b/forum/views/auth.py
index f9d299d29da230eda99184d9a8f04f0b263806f4..3d615750e0d085599fdb1d652a10aba0d9301d8e 100644 (file)
--- a/forum/views/auth.py
+++ b/forum/views/auth.py
@@ -169,10 +169,7 @@ def external_register(request):
             del request.session['assoc_key']
             del request.session['auth_provider']
 
             del request.session['assoc_key']
             del request.session['auth_provider']
 

-            if user_.email_isvalid:
-                return login_and_forward(request, user_)
-            else:
-                return HttpResponseRedirect(reverse('index'))
+            return login_and_forward(request, user_)

     else:
         provider_class = AUTH_PROVIDERS[request.session['auth_provider']].consumer
         user_data = provider_class.get_user_data(request.session['assoc_key'])
     else:
         provider_class = AUTH_PROVIDERS[request.session['auth_provider']].consumer
         user_data = provider_class.get_user_data(request.session['assoc_key'])

diff --git a/forum_modules/openidauth/authentication.py b/forum_modules/openidauth/authentication.py
index 9163b5c5227d73c24a3b1d6a233e82fb363c3408..dfd337f4c1056b6bf274f21b6343f909b1c96ce7 100644 (file)
--- a/forum_modules/openidauth/authentication.py
+++ b/forum_modules/openidauth/authentication.py
@@ -47,8 +47,8 @@ class MyOpenIdAuthConsumer(OpenIdAbstractAuthConsumer):
     dataype2ax_schema = {
         #'username': ('http://schema.openid.net/namePerson/friendly', 'friendly'),
         'email': 'http://schema.openid.net/contact/email',
     dataype2ax_schema = {
         #'username': ('http://schema.openid.net/namePerson/friendly', 'friendly'),
         'email': 'http://schema.openid.net/contact/email',

-        'web': 'http://schema.openid.net/contact/web/default',
-        'birthdate': ('http://schema.openid.net/birthDate', 'birthDate'),
+        #'web': 'http://schema.openid.net/contact/web/default',
+        #'birthdate': ('http://schema.openid.net/birthDate', 'birthDate'),

     }
 
     def get_user_url(self, request):
     }
 
     def get_user_url(self, request):

diff --git a/forum_modules/openidauth/consumer.py b/forum_modules/openidauth/consumer.py
index 22e83d27dfafdb2d1647fcb610d9e7f0e7d4e614..cc1e97d78ef30c75bef87fd9258d623efd3fd5eb 100644 (file)
--- a/forum_modules/openidauth/consumer.py
+++ b/forum_modules/openidauth/consumer.py
@@ -18,12 +18,12 @@ from store import OsqaOpenIDStore
 class OpenIdAbstractAuthConsumer(AuthenticationConsumer):
 
     dataype2ax_schema = {
 class OpenIdAbstractAuthConsumer(AuthenticationConsumer):
 
     dataype2ax_schema = {

-        'username': 'http://axschema.org/namePerson/friendly',
+        #'username': 'http://axschema.org/namePerson/friendly',

         'email': 'http://axschema.org/contact/email',
         'email': 'http://axschema.org/contact/email',

-        'web': 'http://axschema.org/contact/web/default',
+        #'web': 'http://axschema.org/contact/web/default',

         #'firstname': 'http://axschema.org/namePerson/first',
         #'lastname': 'http://axschema.org/namePerson/last',
         #'firstname': 'http://axschema.org/namePerson/first',
         #'lastname': 'http://axschema.org/namePerson/last',

-        'birthdate': 'http://axschema.org/birthDate',
+        #'birthdate': 'http://axschema.org/birthDate',

     }
 
     def get_user_url(self, request):
     }
 
     def get_user_url(self, request):


@@ -107,19 +107,12 @@ class OpenIdAbstractAuthConsumer(AuthenticationConsumer):
                         for n, s in axargs.items() if s in ax_schema2data_type
                     ])
 
                         for n, s in axargs.items() if s in ax_schema2data_type
                     ])
 

-                    #available_data = dict([
-                    #    (t, axargs["value.%s.1" % s]) for t, s in available_types.items()
-                    #])
-
-                    #print available_data
-                    
-
-                    #email = ax.getExtensionArgs()['value.ext0.1']
-                    #username = ax.getExtensionArgs()['value.ext0.2']
+                    available_data = dict([
+                        (t, axargs["value.%s.1" % s]) for t, s in available_types.items()
+                    ])

                     
                     request.session['auth_consumer_data'] = {
                     
                     request.session['auth_consumer_data'] = {

-                        'email': '',
-                        'username': ''
+                        'email': available_data.get('email', None),

                     }
 
                 except Exception, e:
                     }
 
                 except Exception, e: