From 2fc1adc3db253c14fb89ae2e7511ce2ea1691b2a Mon Sep 17 00:00:00 2001 From: Loic Baron Date: Mon, 14 Apr 2014 14:32:23 +0200 Subject: [PATCH] Registration: Check if user exists filter on user_hrn is quicker than the full list of users --- portal/registrationview.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) 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.') -- 2.43.0