[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