summaryrefslogtreecommitdiff
path: root/modules/user
diff options
context:
space:
mode:
authorDavid Rothstein <drothstein@gmail.com>2013-12-25 18:20:59 -0500
committerDavid Rothstein <drothstein@gmail.com>2013-12-25 18:20:59 -0500
commit1f3e78c2dd94df1d17d4273687cb7dc78780d783 (patch)
tree4e2333576c8c62014094d131bc2b084f2f809182 /modules/user
parent63312113a25ce9ad629c6771bef1fa0767955c12 (diff)
downloadbrdo-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.module2
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'])));
}