[wp-trac] [WordPress Trac] #18176: $wpdb should store the database engine and request this of drop-ins

WordPress Trac wp-trac at lists.automattic.com
Thu Jul 21 22:17:17 UTC 2011


#18176: $wpdb should store the database engine and request this of drop-ins
--------------------------+------------------
 Reporter:  nacin         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.3
Component:  Database      |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------

Comment (by nacin):

 Replying to [comment:4 Otto42]:
 > Nacin: don't set the $engine variable there, set it in the wpdb class's
 constructor. If there's not one, make one. Then subclasses only get it if
 they call the parent constructor.

 Thanks for the idea. In that case, we should set it in db_connect(). That
 method is the first one which calls mysql_connect(). Anyone not using
 MySQL will *need* to override that (the constructor is a bit agnostic).

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


More information about the wp-trac mailing list