Make WordPress Core

Opened 15 years ago

Closed 15 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 15 years ago.

Download all attachments as: .zip

Change History (5)

#1 @dd32
15 years ago

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

#2 @solarissmoke
15 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
15 years ago

#3 @nacin
15 years ago

  • Milestone changed from Future Release to 3.2

#4 @dd32
15 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.