[wp-trac] [WordPress Trac] #36224: WP_Taxonomy class
WordPress Trac
noreply at wordpress.org
Thu Oct 6 16:34:21 UTC 2016
#36224: WP_Taxonomy class
-------------------------------------------------+-------------------------
Reporter: swissspidy | Owner: swissspidy
Type: enhancement | Status: assigned
Priority: normal | Milestone: 4.7
Component: Taxonomy | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests dev- | Focuses:
feedback 4.7-early |
-------------------------------------------------+-------------------------
Comment (by boonebgorges):
> Mostly on twitter. tl;dr: final keyword, no interface, mutable, should
use __get() and __set() instead of plenty of properties and set_props().
It seems to me that the goal of this ticket, and the `WP_Post_Type`
ticket, is to bring a small amount of sanity to what is currently a free-
for-all `stdClass` fracas. We are not attempting to impose any actual OOP
architecture. We *should* impose some architecture on it in the future,
but it doesn't necessarily have to happen all at once, and I don't see
that anything in the proposed patch affects our ability to do it down the
road. Backward compatibility will probably always require that existing
properties be mutable, whether or not they're hidden behind `__set()`. And
`final` prevents us from having to make any decisions about the purpose
and design of the interface for the time being.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36224#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list