This is a bit inefficient

       $result = pager_query("SELECT * FROM {users} WHERE LOWER(name) LIKE LOWER('%%%s%%')", 15, 0, NULL, $keys);
        while ($account = db_fetch_object($result)) {
          $find[] = array('title' => $account->name, 'link' => url('user/'. $account->uid, array('absolute' => TRUE)));
        }

The same patch applies to 6.x-dev

CommentFileSizeAuthor
user_search.patch697 byteslyricnz

Comments

lyricnz’s picture

Status: Active » Needs review

To "code needs review"

lyricnz’s picture

I know this probably doesn't make a huge difference, but it does save up to 30KB of php<-->mysql per query (15 rows of 2KB)

killes@www.drop.org’s picture

Version: 5.1 » 6.x-dev
Status: Needs review » Reviewed & tested by the community

this should be good to go.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks! Send us more patches. :-)

m3avrck’s picture

Version: 6.x-dev » 5.x-dev
Status: Fixed » Reviewed & tested by the community

Can 5 get some love to??

drumm’s picture

Sure, 5 can get some love.

Committed to 5.

drumm’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)