[wp-trac] [WordPress Trac] #20838: Don't populate $col_info in production

WordPress Trac wp-trac at lists.automattic.com
Mon Jun 18 18:41:57 UTC 2012


#20838: Don't populate $col_info in production
-------------------------+--------------------
 Reporter:  GrahamKelly  |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.5
Component:  Database     |     Version:  3.3.2
 Severity:  normal       |  Resolution:
 Keywords:  needs-patch  |
-------------------------+--------------------
Changes (by nacin):

 * keywords:  has-patch => needs-patch
 * milestone:  Awaiting Review => 3.5


Comment:

 This could be an easy lazy-loading win.

 We could use a magic getter to handle someone accessing col_info directly,
 then make get_col_info() act on `$this->result` directly. This would
 prevent mysql_fetch_field() from firing unless someone needs it.

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


More information about the wp-trac mailing list