diff options
author | David Rothstein <drothstein@gmail.com> | 2013-12-25 18:20:59 -0500 |
---|---|---|
committer | David Rothstein <drothstein@gmail.com> | 2013-12-25 18:20:59 -0500 |
commit | 1f3e78c2dd94df1d17d4273687cb7dc78780d783 (patch) | |
tree | 4e2333576c8c62014094d131bc2b084f2f809182 /modules/user | |
parent | 63312113a25ce9ad629c6771bef1fa0767955c12 (diff) | |
download | brdo-1f3e78c2dd94df1d17d4273687cb7dc78780d783.tar.gz brdo-1f3e78c2dd94df1d17d4273687cb7dc78780d783.tar.bz2 |
Issue #1996644 by apkwilson, abghosh82 | joachim: User_login_name_validate() uses isset() where other user login form validation handlers use !empty().
Diffstat (limited to 'modules/user')
-rw-r--r-- | modules/user/user.module | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/user/user.module b/modules/user/user.module index 305fec80c..cbb2d00f6 100644 --- a/modules/user/user.module +++ b/modules/user/user.module @@ -2122,7 +2122,7 @@ function user_login_default_validators() { * A FAPI validate handler. Sets an error if supplied username has been blocked. */ function user_login_name_validate($form, &$form_state) { - if (isset($form_state['values']['name']) && user_is_blocked($form_state['values']['name'])) { + if (!empty($form_state['values']['name']) && user_is_blocked($form_state['values']['name'])) { // Blocked in user administration. form_set_error('name', t('The username %name has not been activated or is blocked.', array('%name' => $form_state['values']['name']))); } |