2.8.5 Multiple primary key Error

[Frumph]

[21-Oct-2009 07:19:31] WordPress database error Multiple primary key defined for query ALTER TABLE wp_options ADD PRIMARY KEY (option_id,blog_id,option_name) made by wp_upgrade, make_db_current_silent, dbDelta

This was given after upgrading to 2.8.5

SQL errors should have high priority for 2.9. Duplicate primary key errors are usually caused by failure to check key type and key existence before issuing an ALTER command.

Frumph, are you able to add steps to reproduce this error?

At first glance, function dbDelta() appears to issue the ALTER command as follows:

if(!(($aindex = array_search($index_string, $indices)) === false)) {
    unset($indices[$aindex]);
}
// For every remaining index specified for the table
foreach ( (array) $indices as $index ) {
    // Push a query line into $cqueries that adds the index to that table
    $cqueries[] = "ALTER TABLE {$table} ADD $index";
}

That logic is extremely deficient if it is being used in production to modify table indices.

After a little more digging, I figured out that some of the indices are handled by function pre_schema_upgrade(), and the code in there does not match the error given by OP. I'm starting to doubt that error could be reproduced.

Just coincidentally, this was fixed by [12217] a few hours ago. That's the first revision containing both pre_schema_upgrade() and the correct SQL syntax. Unfortunately, that revision introduced a logic error affecting one of the 2.9-rare schemas, so I'd like to see that patched before this ticket closes.

See also [11883].

(In [12232]) Fix db version logic error. Props miqrogroove. fixes #10994