[wp-trac] [WordPress Trac] #10667: Integration of Search API into core

WordPress Trac wp-trac at lists.automattic.com
Fri Feb 12 16:47:50 UTC 2010


#10667: Integration of Search API into core
----------------------------+-----------------------------------------------
 Reporter:  jshreve         |       Owner:             
     Type:  task (blessed)  |      Status:  new        
 Priority:  normal          |   Milestone:  3.0        
Component:  General         |     Version:  2.9        
 Severity:  normal          |    Keywords:  needs-patch
----------------------------+-----------------------------------------------

Comment(by andy):

 I think I like it better as a filter. I wrote it that way first but I'm
 not sure why I didn't keep it that way.

 Replying to [comment:12 filosofo]:
 > I don't know why you'd want to replace the class in only one place,
 particularly since many themes call query_posts() directly.

 For example, a search plugin could use this. That would probably only want
 to replace/extend the core instance and only when isset($_GET['s']).

 Sometimes you're going to want to ignore the core posts query results so
 you might as well bypass all of WP_Query::get_posts. Heck, why not just
 stub it out in WP::main. That's why I showed up wanting to be able to
 replace or extend these classes.

 While I'm at it, I'm thinking of how to fix wpdb so it can be extended.
 HyperDB is a PITA to keep synced with wpdb. It would be better if it
 extended wpdb.

 So this should be two new tickets then.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10667#comment:13>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list