diff options
| author | Dries Buytaert <dries@buytaert.net> | 2006-10-23 06:21:58 +0000 |
|---|---|---|
| committer | Dries Buytaert <dries@buytaert.net> | 2006-10-23 06:21:58 +0000 |
| commit | 905a679c55cc1fe0d3c64584295554ec86ea4ebf (patch) | |
| tree | c5f21978c4d96f57b6b030aae42094c5cc42c396 /modules | |
| parent | e070e83238ff8007bf63b51f6bbf15d51c3b4827 (diff) | |
| download | brdo-905a679c55cc1fe0d3c64584295554ec86ea4ebf.tar.gz brdo-905a679c55cc1fe0d3c64584295554ec86ea4ebf.tar.bz2 | |
- Patch #89964 by jvandyk: improved performance of user_is_blocked() by removing a redundant query.
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/user/user.module | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/modules/user/user.module b/modules/user/user.module index 48b0b2ee6..ef6a5b8fb 100644 --- a/modules/user/user.module +++ b/modules/user/user.module @@ -375,10 +375,9 @@ function user_access($string, $account = NULL) { * @return boolean TRUE for blocked users, FALSE for active */ function user_is_blocked($name) { - $allow = db_fetch_object(db_query("SELECT * FROM {users} WHERE status = 1 AND name = LOWER('%s')", $name)); - $deny = db_fetch_object(db_query("SELECT * FROM {users} WHERE status = 0 AND name = LOWER('%s')", $name)); + $deny = db_fetch_object(db_query("SELECT name FROM {users} WHERE status = 0 AND name = LOWER('%s')", $name)); - return $deny && !$allow; + return $deny; } function user_fields() { |