[wp-trac] [WordPress Trac] #14162: Introduce WP_Term class

WordPress Trac noreply at wordpress.org
Fri Oct 9 21:27:32 UTC 2015


#14162: Introduce WP_Term class
--------------------------------------+---------------------------
 Reporter:  scribu                    |       Owner:  DrewAPicture
     Type:  enhancement               |      Status:  reviewing
 Priority:  normal                    |   Milestone:  4.4
Component:  Taxonomy                  |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  needs-patch dev-feedback  |     Focuses:
--------------------------------------+---------------------------
Changes (by boonebgorges):

 * owner:  boonebgorges => DrewAPicture
 * status:  assigned => reviewing


Comment:

 [attachment:14162.diff] refreshes the patch. It also fixes a small bit of
 logic in the inline term splitting that happens when a non-matching
 taxonomy is passed to `get_term()` - this was the cause for at least one
 of the test failures. I also streamlined the way sanitization works a
 little bit.

 I looked over the codebase to see if we were overloading `$term` objects
 anywhere, but couldn't find anything. So I feel fairly confident about
 what we've currently got. Let's drop it in and see if the world explodes.

 DrewAPicture, can I ask you for a docs review?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/14162#comment:31>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list