Prefixing of the temporary database tables was introduced in commit http://drupal.org/cvs?commit=20894, but a call to db_prefix_tables() was overlooked.

Patch to add a db_prefix_tables() call in db_query_temporary() to includes/database.mysql.inc and database.pgsql.inc

CommentFileSizeAuthor
#1 db-temp-prefix_0.patch1.63 KBasimmonds
db-temp-prefix.patch1.37 KBasimmonds

Comments

asimmonds’s picture

asimmonds commented
StatusFileSize
new db-temp-prefix_0.patch1.63 KB

Should probably remove the db_prefix_tables() from the line above this one, no use calling db_prefix_tables() twice.

Cvbge’s picture

Cvbge commented

Please see http://drupal.org/node/36352 - prefixing temporary tables is questionable.
I'd prefer to drop prefixing the tables, as I don't see how would it help.

asimmonds’s picture

asimmonds commented

Whoops, sorry I didn't see that issue. Steven introduced the change, so I suppose it's up to him if it stays, but you give a pretty good argument.

asimmonds’s picture

asimmonds commented
Status: Needs review » Fixed

Fixed by:
http://drupal.org/node/36352#comment-55667

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)