diff options
| author | Angie Byron <webchick@24967.no-reply.drupal.org> | 2009-05-16 15:23:16 +0000 |
|---|---|---|
| committer | Angie Byron <webchick@24967.no-reply.drupal.org> | 2009-05-16 15:23:16 +0000 |
| commit | f577c125e84bc3a1f30bcc40105788d0547a534a (patch) | |
| tree | 87654a9fbf162a10ffcb21c4c32289743687b8cd /modules/profile/profile.module | |
| parent | 8b63d832de0c708836393bdf83fffe8bf10c2a3f (diff) | |
| download | brdo-f577c125e84bc3a1f30bcc40105788d0547a534a.tar.gz brdo-f577c125e84bc3a1f30bcc40105788d0547a534a.tar.bz2 | |
#196862 by Damien Tournoud, et al: Replace COUNT(*) queries with SELECT 1 ... LIMIT 1 queries when all that's required is a check for whether rows exist.
Diffstat (limited to 'modules/profile/profile.module')
| -rw-r--r-- | modules/profile/profile.module | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/modules/profile/profile.module b/modules/profile/profile.module index 919910746..893e63dad 100644 --- a/modules/profile/profile.module +++ b/modules/profile/profile.module @@ -511,7 +511,8 @@ function profile_category_access($account, $category) { return TRUE; } else { - return user_edit_access($account) && db_result(db_query("SELECT COUNT(*) FROM {profile_field} WHERE category = '%s' AND visibility <> %d", $category, PROFILE_HIDDEN)); + $category_visible = (bool) db_query_range('SELECT 1 FROM {profile_field} WHERE category = :category AND visibility <> :visibility', array(':category' => $category, ':visibility' => PROFILE_HIDDEN), 0, 1)->fetchField(); + return user_edit_access($account) && $category_visible; } } |