WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Last modified 7 years ago

#20352 new enhancement

Use str_getcsv() to parse search terms in WP_Query

Reported by: scribu Owned by:
Priority: normal Milestone:
Component: Query Version:
Severity: normal Keywords: has-patch needs-refresh needs-unit-tests
Cc: Focuses:

Description (last modified by scribu)

We use an ​ugly regex to split search terms like these:

term1 term2 "exact match"

We could use str_getcsv() instead, for better readability.

Attachments (1)

20352.diff​ (1.7 KB ) - added by scribu 14 years ago.

Download all attachments as: .zip

Change History (8)

#1 @scribu
14 years ago

kurtpayne mentioned in IRC a PHP 5.2 implementation for str_getcsv():

​http://unit-test.trac.wordpress.org/browser/wp-testlib/utils.php#L326

#2 @scribu
14 years ago

  • Description modified (diff)

@scribu
14 years ago

#3 @scribu
14 years ago

  • Keywords has-patch added

First pass.

#4 @kurtpayne
14 years ago

  • Cc kpayne@… added

#5 @nacin
13 years ago

  • Component General → Query

#6 @wonderboymusic
12 years ago

  • Keywords needs-refresh needs-unit-tests added
  • Milestone Awaiting Review → Future Release

Not sure this is the best solution, but we should consider something less nasty

#7 @chriscct7
11 years ago

  • Severity minor → normal
Note: See TracTickets for help on using tickets.