[wp-trac] [WordPress Trac] #30261: Split all existing shared taxonomy terms on WP upgrade

[WordPress Trac]

#30261: Split all existing shared taxonomy terms on WP upgrade
-------------------------------------+---------------------------
 Reporter:  boonebgorges             |       Owner:  boonebgorges
     Type:  enhancement              |      Status:  reopened
 Priority:  high                     |   Milestone:  4.3
Component:  Taxonomy                 |     Version:
 Severity:  blocker                  |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+---------------------------
Changes (by Chouby):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 In [attachment:30261.17.diff], I propose to remove the call to
 `wp_suspend_cache_invalidation()` as I noticed during my tests that it
 introduces a conflict with Polylang. That's would not a big deal for
 Polylang users as it just leaves unused DB entries (so nothing visible to
 the user), but I have no idea of what kind of other conflict it could
 introduce in other plugins using the `split_shared_term` hook.

 I guess that this call was introduced for performances reasons. Now that
 the split is done by batches of 20 terms, it may be less important to save
 time than avoiding potential conflicts.

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