[wp-trac] [WordPress Trac] #18986: Link categories filter doesn't show all categories
WordPress Trac
wp-trac at lists.automattic.com
Fri Oct 21 00:30:46 UTC 2011
#18986: Link categories filter doesn't show all categories
--------------------------+-----------------------
Reporter: ocean90 | Owner: ryan
Type: defect (bug) | Status: reopened
Priority: high | Milestone: 3.3
Component: Taxonomy | Version: 3.3
Severity: major | Resolution:
Keywords: has-patch |
--------------------------+-----------------------
Comment (by nacin):
[attachment:18986.diff] takes a step back and approaches this from a
different perspective.
Essentially, link_category should never be using
_update_post_term_count(). In 3.2, it was using the generic handler, which
was inline. In 3.3, all taxonomies by default now use
_update_post_term_count(). This includes not only link_category, but all
custom taxonomies. This will change how custom taxonomies attached to
posts are counted, assuming they don't specify an update_count_callback. I
always forget to, so perhaps this would normally be for the better for
developers in general, but it is technically a backwards incompatible
change that will make counts drop pretty quickly.
(Reference thus far: [18783/trunk/wp-includes/taxonomy.php], via #17548.)
So I think what we need to do is create a new generic handler,
_update_generic_term_count(). This does restore the original behavior that
was changed in #17548, but it doesn't do so at the expense of all other
taxonomies.
Note that the patch makes this handler the default callback. I don't know
if that's the best idea, given the comments in #17548. So it sounds like
maybe we need to do some detection on $object_types when choosing our
default callback. If it's nothing but post types, then use
_update_post_term_count(), otherwise, we'll need to use the generic
handler.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/18986#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list