Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#17059 closed enhancement (fixed)

Theme searching is case sensitive

Reported by: dd32's profile dd32 Owned by:
Milestone: 3.2 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

When you're on the available themes, and you attempt to search the themes, you'll find it's case sensitive.

'wordcamp' may return 0 results, whilst 'WordCamp' will return 10.

Attachments (1)

17059.diff (649 bytes) - added by solarissmoke 13 years ago.

Download all attachments as: .zip

Change History (5)

#1 @dd32
13 years ago

Extra clarification: This only applies to searching builtin themes, not themes to install.

#2 @solarissmoke
13 years ago

  • Keywords has-patch added; needs-patch removed

Only the bit that searches tags is case sensitive. Searching the title, description etc uses stripos() and is fine. But we may as well strtolower() everything as it comes in, I think.

@solarissmoke
13 years ago

#3 @nacin
13 years ago

  • Milestone changed from Future Release to 3.2

#4 @dd32
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [17691]) Ensure all theme searching is done case-insensitive. Props solarissmoke for initial path. Fixes #17059

Note: See TracTickets for help on using tickets.