wpmu-terms.php fails if there are more terms in the blog's terms table than there are in wp_sitecategories

[mkanat]

fix_blog_terms in wpmu-terms does this:

$maxterm = $wpdb->get_var( "SELECT max(cat_ID) FROM {$wpdb->sitecategories}" )

Then later it does this:

$term_id = $row->term_id + mt_rand( $maxterm+100, $maxterm+4000 );

However, if there are thousands of terms in the blog's terms table, and only a few terms in wp_sitecategories, this will result in term_ids that conflict with other term_ids in $wpdb->terms.

The simple (though probably not ideal) solution is to change the $maxterm line to:

$maxterm = max($wpdb->get_var( "SELECT max(cat_ID) FROM {$wpdb->sitecategories}" ),

$wpdb->get_var( "SELECT max(cat_ID) FROM {$wpdb->terms}" ) );

[mkanat]

By the way, the reason this bug is marked as "major" severity is that running wpmu-terms.php on a blog with this problem causes the terms table to basically be fubar'ed beyond recovery.

[nacin]

wpmu-terms will not be in WordPress 3.0.

[mkanat]

Patch, v1

[mkanat]

I discovered that my code above was slightly wrong. I have a working version now with the attached patch.

[nacin]

Closing as wontfix due to this page not appearing in 3.0. Leaving the milestone attached however for record keeping purposes.