Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

- #36255: Fix db_query_temporary() in pgsql 

- Prefix temporary table names (helps on restricted hosts)
4.7.x
Steven Wittens 2005-11-04 13:02:51 +00:00
parent 132517a8cb
commit 2ad97f0277
4 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
includes/database.mysql.inc
View File

@ -268,8 +268,8 @@ function db_query_range($query) {
* printf() syntax. Instead of a variable number of query arguments, you may
* also pass a single array containing the query arguments.
* @param $table
* The name of the temporary table to select into. This name will not be
* prefixed as there is no risk of collision.
* The name of the temporary table to select into. This name will be
* prefixed for compatibility reasons (even though there is no risk of collision)@.
* @return
* A database query result resource, or FALSE if the query was not executed
* correctly.
@ -278,7 +278,7 @@ function db_query_temporary($query) {
$args = func_get_args();
$tablename = array_pop($args);
$query = preg_replace('/^SELECT/i', 'CREATE TEMPORARY TABLE '. $tablename .' SELECT', db_prefix_tables($query));
$query = preg_replace('/^SELECT/i', 'CREATE TEMPORARY TABLE {'. $tablename .'} SELECT', db_prefix_tables($query));
if (count($args) > 1) {
// Check for array (alternative syntax).
if (is_array($args[1])) {

6
includes/database.pgsql.inc
View File

@ -253,8 +253,8 @@ function db_query_range($query) {
* printf() syntax. Instead of a variable number of query arguments, you may
* also pass a single array containing the query arguments.
* @param $table
* The name of the temporary table to select into. This name will not be
* prefixed as there is no risk of collision.
* The name of the temporary table to select into. This name will be
* prefixed for compatibility reasons (there is no risk of collision).
* @return
* A database query result resource, or FALSE if the query was not executed
* correctly.
@ -263,7 +263,7 @@ function db_query_temporary($query) {
$args = func_get_args();
$tablename = array_pop($args);
$query = preg_replace('/^SELECT/i', 'CREATE TEMPORARY TABLE '. $tablename .' AS', db_prefix_tables($query));
$query = preg_replace('/^SELECT/i', 'CREATE TEMPORARY TABLE {'. $tablename .'} AS SELECT', db_prefix_tables($query));
if (count($args) > 1) {
// Check for array (alternative syntax).
if (is_array($args[1])) {

8
modules/search.module
View File

@ -775,7 +775,7 @@ function do_search($keywords, $type, $join1 = '', $where1 = '1', $arguments1 = a
$result = db_query_temporary("SELECT i.type, i.sid, SUM(i.score * t.count) AS relevance, COUNT(*) AS matches FROM {search_index} i INNER JOIN {search_total} t ON i.word = t.word $join1 WHERE $conditions GROUP BY i.type, i.sid HAVING matches >= %d", $arguments, 'temp_search_sids');
// Calculate maximum relevance, to normalize it
$normalize = db_result(db_query('SELECT MAX(relevance) FROM temp_search_sids'));
$normalize = db_result(db_query('SELECT MAX(relevance) FROM {temp_search_sids}'));
if (!$normalize) {
return array();
}
@ -784,14 +784,14 @@ function do_search($keywords, $type, $join1 = '', $where1 = '1', $arguments1 = a
// Second pass: only keep items that match the complicated keywords conditions (phrase search, negative keywords, ...)
$conditions = '('. $query[0] .')';
$arguments = array_merge($arguments2, $query[1]);
$result = db_query_temporary("SELECT i.type, i.sid, $select2 FROM temp_search_sids i INNER JOIN {search_dataset} d ON i.sid = d.sid AND i.type = d.type $join2 WHERE $conditions ORDER BY score DESC", $arguments, 'temp_search_results');
if (($count = db_result(db_query('SELECT COUNT(*) FROM temp_search_results'))) == 0) {
$result = db_query_temporary("SELECT i.type, i.sid, $select2 FROM {temp_search_sids} i INNER JOIN {search_dataset} d ON i.sid = d.sid AND i.type = d.type $join2 WHERE $conditions ORDER BY score DESC", $arguments, 'temp_search_results');
if (($count = db_result(db_query('SELECT COUNT(*) FROM {temp_search_results}'))) == 0) {
return array();
}
$count_query = "SELECT $count";
// Do actual search query
$result = pager_query("SELECT * FROM temp_search_results", 10, 0, $count_query, $arguments);
$result = pager_query("SELECT * FROM {temp_search_results}", 10, 0, $count_query, $arguments);
$results = array();
while ($item = db_fetch_object($result)) {
$results[] = $item->sid;

8
modules/search/search.module
View File

@ -775,7 +775,7 @@ function do_search($keywords, $type, $join1 = '', $where1 = '1', $arguments1 = a
$result = db_query_temporary("SELECT i.type, i.sid, SUM(i.score * t.count) AS relevance, COUNT(*) AS matches FROM {search_index} i INNER JOIN {search_total} t ON i.word = t.word $join1 WHERE $conditions GROUP BY i.type, i.sid HAVING matches >= %d", $arguments, 'temp_search_sids');
// Calculate maximum relevance, to normalize it
$normalize = db_result(db_query('SELECT MAX(relevance) FROM temp_search_sids'));
$normalize = db_result(db_query('SELECT MAX(relevance) FROM {temp_search_sids}'));
if (!$normalize) {
return array();
}
@ -784,14 +784,14 @@ function do_search($keywords, $type, $join1 = '', $where1 = '1', $arguments1 = a
// Second pass: only keep items that match the complicated keywords conditions (phrase search, negative keywords, ...)
$conditions = '('. $query[0] .')';
$arguments = array_merge($arguments2, $query[1]);
$result = db_query_temporary("SELECT i.type, i.sid, $select2 FROM temp_search_sids i INNER JOIN {search_dataset} d ON i.sid = d.sid AND i.type = d.type $join2 WHERE $conditions ORDER BY score DESC", $arguments, 'temp_search_results');
if (($count = db_result(db_query('SELECT COUNT(*) FROM temp_search_results'))) == 0) {
$result = db_query_temporary("SELECT i.type, i.sid, $select2 FROM {temp_search_sids} i INNER JOIN {search_dataset} d ON i.sid = d.sid AND i.type = d.type $join2 WHERE $conditions ORDER BY score DESC", $arguments, 'temp_search_results');
if (($count = db_result(db_query('SELECT COUNT(*) FROM {temp_search_results}'))) == 0) {
return array();
}
$count_query = "SELECT $count";
// Do actual search query
$result = pager_query("SELECT * FROM temp_search_results", 10, 0, $count_query, $arguments);
$result = pager_query("SELECT * FROM {temp_search_results}", 10, 0, $count_query, $arguments);
$results = array();
while ($item = db_fetch_object($result)) {
$results[] = $item->sid;