[wp-trac] [WordPress Trac] #18510: Class properties dbpassword, dbname, dbhost and dbh are undefined in wpdb

WordPress Trac wp-trac at lists.automattic.com
Thu Aug 25 23:47:43 UTC 2011


#18510: Class properties dbpassword, dbname, dbhost and dbh are undefined in wpdb
------------------------------------+------------------------------
 Reporter:  joelhardi               |       Owner:
     Type:  enhancement             |      Status:  new
 Priority:  normal                  |   Milestone:  Awaiting Review
Component:  Database                |     Version:  3.3
 Severity:  minor                   |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+------------------------------

Comment (by joelhardi):

 Sure ... I'm just saying you're more likely to get a response if you
 submit a new ticket with a full patch. Adding as a comment on a mostly
 unrelated ticket is usually not going to get you any traction. It works to
 your advantage and keeps discussion here from spiraling off topic. My
 ticket may go nowhere whereas yours might. Or, if my conservative patch is
 applied, this ticket is closed and your idea/work is buried.

 My ticket/patch doesn't change any functionality, yours does, that's why
 they're not the same thing. My goal is to fill in some missing
 documentation, yours is to move WP to PHP5 (or something like that).

 You have to understand that enforcing visibility will break many plugins,
 the issue isn't PHP4 vs. PHP5. !WordPress is all about having things "just
 work" and generally not about breakage for its own sake. A change to the
 API like yours that removes access has to be phased in gradually and
 broadcast to the community. (For example, one strategy would be to add an
 accessor method 12 months before enforcing private access to a member
 property.) Anyway ... this is the discussion I was trying to avoid so I'll
 cut it off there. Of course it's up to you, just trying to advise you
 about how things normally work with core.

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


More information about the wp-trac mailing list