[wp-trac] [WordPress Trac] #11799: WPDB code improvements

WordPress Trac wp-trac at lists.automattic.com
Fri Jan 8 17:54:56 UTC 2010


#11799: WPDB code improvements
-------------------------+--------------------------------------------------
 Reporter:  hakre        |       Owner:  ryan     
     Type:  enhancement  |      Status:  new      
 Priority:  normal       |   Milestone:  3.0      
Component:  Database     |     Version:  3.0      
 Severity:  normal       |    Keywords:  has-patch
-------------------------+--------------------------------------------------

Comment(by Denis-de-Bernardy):

 isn't this missing a delimiter?

 {{{
 if ( preg_match( '(^\s*([a-z]{2,})\s+)i', $query, $matches ) )
 }}}

 also, I don't think it'll properly deal with:

 {{{
 query("
 # some comments for debugging using SAVEQUERIES
 INSERT INTO ...
 ");
 }}}

 but nor does the current version of the class, so that's probably another
 ticket...

 also while we're at it, for reference, the code in wpmu does not deal with
 a tabbed update statements properly. (but yours does, since it follows the
 UPDATE with a \s instead of a space in the regexp.)

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


More information about the wp-trac mailing list