diff options
author | cash <cash@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2010-11-15 12:41:26 +0000 |
---|---|---|
committer | cash <cash@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2010-11-15 12:41:26 +0000 |
commit | 624cab6eff69d43c70cacedf8fa2d8d9fa64fabd (patch) | |
tree | 43fa5752c192a6ebf50a4a082db715097f335240 /mod/uservalidationbyemail/start.php | |
parent | 481a5ecfe6b17836d47b8c89eabf346f941bbc3b (diff) | |
download | elgg-624cab6eff69d43c70cacedf8fa2d8d9fa64fabd.tar.gz elgg-624cab6eff69d43c70cacedf8fa2d8d9fa64fabd.tar.bz2 |
Refs #2573 fixes validation issue for users registered with validation turned off. Still need to write an upgrade script since validation status wasn't set on unvalidated users before this
git-svn-id: http://code.elgg.org/elgg/trunk@7320 36083f99-b078-4883-b0ff-0f9b5a30f544
Diffstat (limited to 'mod/uservalidationbyemail/start.php')
-rw-r--r-- | mod/uservalidationbyemail/start.php | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/mod/uservalidationbyemail/start.php b/mod/uservalidationbyemail/start.php index 8c91c5a1f..8f5888679 100644 --- a/mod/uservalidationbyemail/start.php +++ b/mod/uservalidationbyemail/start.php @@ -51,21 +51,22 @@ function uservalidationbyemail_init() { /** * Disables a user upon registration. * - * @param unknown_type $hook - * @param unknown_type $type - * @param unknown_type $value - * @param unknown_type $params + * @param string $hook + * @param string $type + * @param bool $value + * @param array $params + * @return bool */ function uservalidationbyemail_disable_new_user($hook, $type, $value, $params) { $user = elgg_get_array_value('user', $params); // no clue what's going on, so don't react. if (!$user instanceof ElggUser) { - return NULL; + return; } // disable user to prevent showing up on the site - // set context to our canEdit() override works + // set context so our canEdit() override works elgg_push_context('uservalidationbyemail_new_user'); $hidden_entities = access_get_show_hidden_status(); access_show_hidden_entities(TRUE); @@ -83,7 +84,7 @@ function uservalidationbyemail_disable_new_user($hook, $type, $value, $params) { elgg_pop_context(); access_show_hidden_entities($hidden_entities); - return TRUE; + return $value; } /** @@ -123,7 +124,7 @@ function uservalidationbyemail_check_auth_attempt($credentials) { access_show_hidden_entities(TRUE); $user = get_user_by_username($username); - if ($user && !$user->validated) { + if ($user && isset($user->validated) && !$user->validated) { // show an error and resend validation email uservalidationbyemail_request_validation($user->guid); access_show_hidden_entities($access_status); |