Merge branch 'QA_3_3'

This commit is contained in:
Marc Delisle
2011-01-12 08:05:40 -05:00
3 changed files with 9 additions and 4 deletions

View File

@@ -135,6 +135,7 @@
- patch #3150164 [structure] Ordering by size gives incorrect results, - patch #3150164 [structure] Ordering by size gives incorrect results,
thanks to Madhura Jayaratne - madhuracj thanks to Madhura Jayaratne - madhuracj
- bug #3153409 [core] 0 row(s) affected - bug #3153409 [core] 0 row(s) affected
- bug #3155842 [core] Edit relational page and page number
3.3.9.0 (2011-01-03) 3.3.9.0 (2011-01-03)
- bug [doc] Fix references to MySQL doc - bug [doc] Fix references to MySQL doc

View File

@@ -392,13 +392,11 @@ function PMA_DBI_insert_id($link = null)
return false; return false;
} }
} }
//$insert_id = mysql_insert_id($link); // If the primary key is BIGINT we get an incorrect result
// if the primary key is BIGINT we get an incorrect result
// (sometimes negative, sometimes positive) // (sometimes negative, sometimes positive)
// and in the present function we don't know if the PK is BIGINT // and in the present function we don't know if the PK is BIGINT
// so better play safe and use LAST_INSERT_ID() // so better play safe and use LAST_INSERT_ID()
// //
// by the way, no problem with mysqli_insert_id()
return PMA_DBI_fetch_value('SELECT LAST_INSERT_ID();', 0, 0, $link); return PMA_DBI_fetch_value('SELECT LAST_INSERT_ID();', 0, 0, $link);
} }

View File

@@ -452,7 +452,13 @@ function PMA_DBI_insert_id($link = '')
return false; return false;
} }
} }
return mysqli_insert_id($link); // When no controluser is defined, using mysqli_insert_id($link)
// does not always return the last insert id due to a mixup with
// the tracking mechanism, but this works:
return PMA_DBI_fetch_value('SELECT LAST_INSERT_ID();', 0, 0, $link);
// Curiously, this problem does not happen with the mysql extension but
// there is another problem with BIGINT primary keys so PMA_DBI_insert_id()
// in the mysql extension also uses this logic.
} }
/** /**