db_last_insert_id

[Jeff Greenberg-3]

I'm confused with the db_last_insert_id function in
database.mysql-common.inc.


The function is:


function db_last_insert_id($table, $field) {
  return db_result(db_query('SELECT LAST_INSERT_ID()'));
}


and I see no reason whatsoever for there to be any parameters, let alone
required ones.

[Joe Pletcher]

The postgres version is:

function db_last_insert_id($table, $field) {
   return db_result(db_query("SELECT CURRVAL('{". db_escape_table
($table) ."}_". db_escape_table($field) ."_seq')"));
}

Joe

On Nov 4, 2009, at 7:07 PM, Jeff Greenberg wrote:

> I'm confused with the db_last_insert_id function in database.mysql-
> common.inc.
>
>
> The function is:
>
>
> function db_last_insert_id($table, $field) {
> return db_result(db_query('SELECT LAST_INSERT_ID()'));
> }
>
>
> and I see no reason whatsoever for there to be any parameters, let  
> alone required ones.
>

[Mlen-Too Wesley]

It works differently in PostgreSQL, see database.pgsql.inc, or here:
http://api.drupal.org/api/function/db_last_insert_id

On Wed, Nov 4, 2009 at 7:07 PM, Jeff Greenberg <jeff@...> wrote:

I'm confused with the db_last_insert_id function in database.mysql-common.inc.


The function is:


function db_last_insert_id($table, $field) {
 return db_result(db_query('SELECT LAST_INSERT_ID()'));
}


and I see no reason whatsoever for there to be any parameters, let alone required ones.

[Jamie Holly]

Not for MySQL, but that function needs to be the same for the other
databases. Look at database.pgsql.inc:

function db_last_insert_id($table, $field) {
  return db_result(db_query("SELECT CURRVAL('{". db_escape_table($table)
."}_". db_escape_table($field) ."_seq')"));
}


Jamie Holly
http://www.intoxination.net 
http://www.hollyit.net



Jeff Greenberg wrote:

> I'm confused with the db_last_insert_id function in
> database.mysql-common.inc.
>
>
> The function is:
>
>
> function db_last_insert_id($table, $field) {
>   return db_result(db_query('SELECT LAST_INSERT_ID()'));
> }
>
>
> and I see no reason whatsoever for there to be any parameters, let alone
> required ones.
>
>
>  

[Jeff Greenberg-3]

Ah, makes complete sense now. I need to become more 'culturally' aware!  
Thanks!