Takenbacon / wowarmory

The World of Warcraft Armory is a vast searchable database of information for World of Warcraft - taken straight from the real servers and presented in a user-friendly interface. Since the Armory pulls its data from the actual game servers, it is the most comprehensive and up-to-date database on the characters, arena teams, guilds, and items of World of Warcraft in existence.
GNU General Public License v3.0
19 stars 10 forks source link

Player search query case sensitive #7

Closed LeGuybrush closed 11 years ago

LeGuybrush commented 11 years ago

This isn't a major issue, but something to note at any rate. When searching specifically for a character named "player", the search query "player" yields no results, whereas "Player" does.

This is a duplicate issue of #1, but apparently the fix did not solve the problem.

LeGuybrush commented 11 years ago

Fixed. class.search.php:664

$current_realm = $db->select("SELECT `guid`, `name`, `class` AS `classId`, `gender` AS `genderId`, `race` AS `raceId`, `level`, `account` FROM `characters` WHERE `name` = '%s'", $this->searchQuery);

should be

$current_realm = $db->select("SELECT `guid`, `name`, `class` AS `classId`, `gender` AS `genderId`, `race` AS `raceId`, `level`, `account` FROM `characters` WHERE `name` = '%s'", $search);
Takenbacon commented 11 years ago

Good catch, i guess i forgot that part as well when i did that change. I still to this day don't fully understand what causes that to happen on only certain setups. I'll just update the file via git.