Ticket #30261: 30261.7.diff

src/wp-admin/includes/admin-filters.php

@@ -100 +100 @@
 // Update hooks.
 add_action( 'admin_init', 'wp_plugin_update_rows' );
 add_action( 'admin_init', 'wp_theme_update_rows'  );
+add_action( 'admin_footer', 'maybe_split_shared_terms' );
 
 add_action( 'admin_notices', 'update_nag',      3  );
 add_action( 'admin_notices', 'maintenance_nag', 10 );

src/wp-admin/includes/schema.php

@@ -496 +496 @@
 
         // 3.5
         'link_manager_enabled' => 0,
+
+        // 4.3
+        'shared_terms_split' => 0,
         );
 
         // 3.3

src/wp-admin/includes/taxonomy.php

@@ -288 +288 @@
 
         return wp_insert_term($tag_name, $taxonomy);
 }
+
+/**
+ * Initialize the shared term splitting routine, if necessary.
+ *
+ * @since 4.3.0
+ */
+function maybe_split_shared_terms() {
+        // Shared terms only need to be split once.
+        if ( get_option( 'shared_terms_split' ) ) {
+                return;
+        }
+
+        // Run only for authors, on normal page loads, to prevent flooding.
+        if ( ! current_user_can( 'edit_posts' ) || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+                return;
+        }
+
+        wp_batch_split_terms();
+}
+
+/**
+ * Splits a batch of shared taxonomy terms.
+ *
+ * @since 4.3.0
+ *
+ * @global wpdb $wpdb WordPress database abstraction object.
+ */
+function wp_batch_split_terms() {
+        global $wpdb;
+
+        // Get a list of shared terms (those with more than one associated row in term_taxonomy).
+        $shared_terms = $wpdb->get_results(
+                "SELECT tt.term_id, t.*, count(*) as term_tt_count FROM {$wpdb->term_taxonomy} tt
+                 LEFT JOIN {$wpdb->terms} t ON t.term_id = tt.term_id
+                 GROUP BY t.term_id
+                 HAVING term_tt_count > 1
+                 LIMIT 20"
+        );
+
+        // No shared terms found? No need to run this script again.
+        if ( empty( $shared_terms ) ) {
+                update_option( 'shared_terms_split', 1 );
+                return;
+        }
+
+        // Rekey shared term array for faster lookups.
+        $_shared_terms = array();
+        foreach ( $shared_terms as $shared_term ) {
+                $term_id = intval( $shared_term->term_id );
+                $_shared_terms[ $term_id ] = $shared_term;
+        }
+        $shared_terms = $_shared_terms;
+
+        // Get term taxonomy data for all shared terms.
+        $shared_term_ids = implode( ',', array_keys( $shared_terms ) );
+        $shared_tts = $wpdb->get_results( "SELECT * FROM {$wpdb->term_taxonomy} WHERE `term_id` IN ({$shared_term_ids})" );
+
+        // Split term data recording is slow, so we do it just once, outside the loop.
+        $suspend = wp_suspend_cache_invalidation( true );
+        $split_term_data = get_option( '_split_terms', array() );
+        $skipped_first_term = $taxonomies = array();
+        foreach ( $shared_tts as $shared_tt ) {
+                $term_id = intval( $shared_tt->term_id );
+
+                // Don't split the first tt belonging to a given term_id.
+                if ( ! isset( $skipped_first_term[ $term_id ] ) ) {
+                        $skipped_first_term[ $term_id ] = 1;
+                        continue;
+                }
+
+                if ( ! isset( $split_term_data[ $term_id ] ) ) {
+                        $split_term_data[ $term_id ] = array();
+                }
+
+                // Keep track of taxonomies whose hierarchies need flushing.
+                if ( ! isset( $taxonomies[ $shared_tt->taxonomy ] ) ) {
+                        $taxonomies[ $shared_tt->taxonomy ] = 1;
+                }
+
+                // Split the term.
+                $split_term_data[ $term_id ][ $shared_tt->taxonomy ] = _split_shared_term( $shared_terms[ $term_id ], $shared_tt, false );
+        }
+
+        // Rebuild the cached hierarchy for each affected taxonomy.
+        foreach ( array_keys( $taxonomies ) as $tax ) {
+                delete_option( "{$tax}_children" );
+                _get_term_hierarchy( $tax );
+        }
+
+        wp_suspend_cache_invalidation( $suspend );
+        update_option( '_split_terms', $split_term_data );
+}

src/wp-admin/includes/upgrade.php

@@ -1505 +1505 @@
                 upgrade_430_fix_comments();
         }
 
-        if ( $wp_current_db_version < 32814 ) {
-                split_all_shared_terms();
-        }
-
         if ( $wp_current_db_version < 33055 && 'utf8mb4' === $wpdb->charset ) {
                 if ( is_multisite() ) {
                         $tables = $wpdb->tables( 'blog' );
@@ -1881 +1877 @@
 }
 
 /**
- * Splits all shared taxonomy terms.
- *
- * @since 4.3.0
- *
- * @global wpdb $wpdb WordPress database abstraction object.
- */
-function split_all_shared_terms() {
-        global $wpdb;
-
-        // Get a list of shared terms (those with more than one associated row in term_taxonomy).
-        $shared_terms = $wpdb->get_results(
-                "SELECT tt.term_id, t.*, count(*) as term_tt_count FROM {$wpdb->term_taxonomy} tt
-                 LEFT JOIN {$wpdb->terms} t ON t.term_id = tt.term_id
-                 GROUP BY t.term_id
-                 HAVING term_tt_count > 1"
-        );
-
-        if ( empty( $shared_terms ) ) {
-                return;
-        }
-
-        // Rekey shared term array for faster lookups.
-        $_shared_terms = array();
-        foreach ( $shared_terms as $shared_term ) {
-                $term_id = intval( $shared_term->term_id );
-                $_shared_terms[ $term_id ] = $shared_term;
-        }
-        $shared_terms = $_shared_terms;
-
-        // Get term taxonomy data for all shared terms.
-        $shared_term_ids = implode( ',', array_keys( $shared_terms ) );
-        $shared_tts = $wpdb->get_results( "SELECT * FROM {$wpdb->term_taxonomy} WHERE `term_id` IN ({$shared_term_ids})" );
-
-        // Split term data recording is slow, so we do it just once, outside the loop.
-        $suspend = wp_suspend_cache_invalidation( true );
-        $split_term_data = get_option( '_split_terms', array() );
-        $skipped_first_term = $taxonomies = array();
-        foreach ( $shared_tts as $shared_tt ) {
-                $term_id = intval( $shared_tt->term_id );
-
-                // Don't split the first tt belonging to a given term_id.
-                if ( ! isset( $skipped_first_term[ $term_id ] ) ) {
-                        $skipped_first_term[ $term_id ] = 1;
-                        continue;
-                }
-
-                if ( ! isset( $split_term_data[ $term_id ] ) ) {
-                        $split_term_data[ $term_id ] = array();
-                }
-
-                // Keep track of taxonomies whose hierarchies need flushing.
-                if ( ! isset( $taxonomies[ $shared_tt->taxonomy ] ) ) {
-                        $taxonomies[ $shared_tt->taxonomy ] = 1;
-                }
-
-                // Split the term.
-                $split_term_data[ $term_id ][ $shared_tt->taxonomy ] = _split_shared_term( $shared_terms[ $term_id ], $shared_tt, false );
-        }
-
-        // Rebuild the cached hierarchy for each affected taxonomy.
-        foreach ( array_keys( $taxonomies ) as $tax ) {
-                delete_option( "{$tax}_children" );
-                _get_term_hierarchy( $tax );
-        }
-
-        wp_suspend_cache_invalidation( $suspend );
-        update_option( '_split_terms', $split_term_data );
-}
-
-/**
  * Retrieve all options as it was for 1.2.
  *
  * @since 1.2.0

src/wp-includes/taxonomy.php

@@ -4249 +4249 @@
                 $term_taxonomy_id = intval( $term_taxonomy->term_taxonomy_id );
         }
 
-        // Don't try to split terms if database schema does not support shared slugs.
-        $current_db_version = get_option( 'db_version' );
-        if ( $current_db_version < 30133 ) {
-                return $term_id;
-        }
-
         // If there are no shared term_taxonomy rows, there's nothing to do here.
         $shared_tt_count = $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(*) FROM $wpdb->term_taxonomy tt WHERE tt.term_id = %d AND tt.term_taxonomy_id != %d", $term_id, $term_taxonomy_id ) );