Fix the account creation form prepopulated fields Change-Id: Iecb5b22b5456b54f61c33cb040a4f932f17f7f76 Signed-off-by: Eric Poirier <eric@eclipse.org>
diff --git a/eclipse.org-common/classes/users/siteLogin.class.php b/eclipse.org-common/classes/users/siteLogin.class.php index cf788b8..b24a44e 100644 --- a/eclipse.org-common/classes/users/siteLogin.class.php +++ b/eclipse.org-common/classes/users/siteLogin.class.php
@@ -763,6 +763,13 @@ } private function _get_default_profile_fields($get_default_values = FALSE){ + + // Making sure we don't have an empty user_uid to avoid pre-populating + // the account creation fields with an empty user_uid + if (empty($this->user_uid)) { + return FALSE; + } + if (empty($this->messages['profile']['danger'])) { $sql = "SELECT /* USE MASTER */ user_org as organization, user_jobtitle as jobtitle, user_bio as bio, user_interests as interests, user_website as website, user_twitter_handle as twitter_handle, user_country as country, user_gender as gender @@ -829,6 +836,9 @@ ); # Validate values + if (empty($fields['user_uid']) || !is_string($fields['user_uid'])) { + $this->messages['profile']['danger'][] = 'Invalid user id<br>'; + } if (!empty($fields['user_website']) && !filter_var($fields['user_website'], FILTER_VALIDATE_URL)) { $this->messages['profile']['danger'][] = 'Invalid website URL<br>'; }