From: Loic Baron <loic.baron@lip6.fr>
Date: Mon, 14 Apr 2014 12:32:23 +0000 (+0200)
Subject: Registration: Check if user exists filter on user_hrn is quicker than the full list... 
X-Git-Tag: myslice-1.1~143
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=2fc1adc3db253c14fb89ae2e7511ce2ea1691b2a;hp=6f6612123b45e1049c03c5f0e58ea780a7bed7ef;p=myslice.git

Registration: Check if user exists filter on user_hrn is quicker than the full list of users
---

diff --git a/portal/registrationview.py b/portal/registrationview.py
index 396d357b..dade361f 100644
--- a/portal/registrationview.py
+++ b/portal/registrationview.py
@@ -82,8 +82,11 @@ class RegistrationView (FreeAccessView, ThemeView):
                 if user_detail['email'] == user_request['email']:
                     errors.append('Email already registered in Manifold. Please provide a new email address.')
             # Does the user exist in sfa? [query is very slow!!]
-            user_query  = Query().get('user').select('user_hrn','user_email')
+            #user_query  = Query().get('user').select('user_hrn','user_email')
+            # XXX Test based on the user_hrn is quick
+            user_query  = Query().get('user').select('user_hrn','user_email').filter_by('user_hrn','==',user_request['user_hrn'])
             user_details_sfa = execute_admin_query(wsgi_request, user_query)
+
             for user in user_details_sfa:
                 if user['user_email'] == user_request['email']:
                     errors.append('Email already registered in SFA registry. Please use another email.')