diff --git a/modules/node/node.module b/modules/node/node.module index dc6532483cfdc8fb330e83f7714a4f27d154ff1d..8ea70f98b15a13faf58c7463e22328259ac22a02 100644 --- a/modules/node/node.module +++ b/modules/node/node.module @@ -1542,16 +1542,10 @@ function node_search_execute($keys = NULL) { // Add the ranking expressions. _node_rankings($query); - // Add a count query. - $inner_query = clone $query; - $count_query = db_select($inner_query->fields('i', array('sid')), NULL, array('target' => 'slave')); - $count_query->addExpression('COUNT(*)'); - $query->setCountQuery($count_query); + // Load results. $find = $query ->limit(10) ->execute(); - - // Load results. $results = array(); foreach ($find as $item) { // Render the node. diff --git a/modules/search/search.api.php b/modules/search/search.api.php index 408cff34fb33dab01e4b2b6b2ad58843a30ed138..21f4174b943e220e6e1504dee4ebf65a6a78359d 100644 --- a/modules/search/search.api.php +++ b/modules/search/search.api.php @@ -177,16 +177,10 @@ function hook_search_execute($keys = NULL) { // Add the ranking expressions. _node_rankings($query); - // Add a count query. - $inner_query = clone $query; - $count_query = db_select($inner_query->fields('i', array('sid'))); - $count_query->addExpression('COUNT(*)'); - $query->setCountQuery($count_query); + // Load results. $find = $query ->limit(10) ->execute(); - - // Load results. $results = array(); foreach ($find as $item) { // Build the node body. diff --git a/modules/search/search.extender.inc b/modules/search/search.extender.inc index 047853a9c2ab8ce7f1c1a83623418685df2f4678..1eae2caa6c629bd5dedb8c919c77b763fdff3030 100644 --- a/modules/search/search.extender.inc +++ b/modules/search/search.extender.inc @@ -407,6 +407,7 @@ public function execute() return new DatabaseStatementEmpty(); } + // Add conditions to query. $this->join('search_dataset', 'd', 'i.sid = d.sid AND i.type = d.type'); $this->condition($this->conditions); @@ -443,4 +444,35 @@ public function execute() return $this->query->execute(); } + + /** + * Build the default count query for SearchQuery. + * + * Since SearchQuery always uses GROUP BY, we can default to a subquery. Also + * adding the same conditions as execute() because countQuery() is called + * first. + */ + public function countQuery() { + // Clone the inner query. + $inner = clone $this->query; + + // Add conditions to query. + $inner->join('search_dataset', 'd', 'i.sid = d.sid AND i.type = d.type'); + $inner->condition($this->conditions); + + // Remove existing fields and expressions, they are not needed for a count + // query. + $fields =& $inner->getFields(); + $fields = array(); + $expressions =& $inner->getExpressions(); + $expressions = array(); + + // Add the sid as the only field and count them as a subquery. + $count = db_select($inner->fields('i', array('sid')), NULL, array('target' => 'slave')); + + // Add the COUNT() expression. + $count->addExpression('COUNT(*)'); + + return $count; + } } diff --git a/modules/search/search.test b/modules/search/search.test index 1a78c9eb46d756aa69f72e577fe28d469932b9d8..0a5e069e3294567ff2f0c52d300af1062c64c321 100644 --- a/modules/search/search.test +++ b/modules/search/search.test @@ -593,6 +593,66 @@ class SearchBlockTestCase extends DrupalWebTestCase { } } +/** + * Tests that searching for a phrase gets the correct page count. + */ +class SearchExactTestCase extends DrupalWebTestCase { + public static function getInfo() { + return array( + 'name' => 'Search engine phrase queries', + 'description' => 'Tests that searching for a phrase gets the correct page count.', + 'group' => 'Search', + ); + } + + function setUp() { + parent::setUp('search'); + } + + /** + * Tests that the correct number of pager links are found for both keywords and phrases. + */ + function testExactQuery() { + // Login with sufficient privileges. + $this->drupalLogin($this->drupalCreateUser(array('create page content', 'search content'))); + + $settings = array( + 'type' => 'page', + 'title' => 'Simple Node', + ); + // Create nodes with exact phrase. + for ($i = 0; $i <= 17; $i++) { + $settings['body'] = array(LANGUAGE_NONE => array(array('value' => 'love pizza'))); + $this->drupalCreateNode($settings); + } + // Create nodes containing keywords. + for ($i = 0; $i <= 17; $i++) { + $settings['body'] = array(LANGUAGE_NONE => array(array('value' => 'love cheesy pizza'))); + $this->drupalCreateNode($settings); + } + + // Update the search index. + module_invoke_all('update_index'); + search_update_totals(); + + // Refresh variables after the treatment. + $this->refreshVariables(); + + // Test that the correct number of pager links are found for keyword search. + $edit = array('keys' => 'love pizza'); + $this->drupalPost('search/node', $edit, t('Search')); + $this->assertLinkByHref('page=1', 0, '2nd page link is found for keyword search.'); + $this->assertLinkByHref('page=2', 0, '3rd page link is found for keyword search.'); + $this->assertLinkByHref('page=3', 0, '4th page link is found for keyword search.'); + $this->assertNoLinkByHref('page=4', '5th page link is not found for keyword search.'); + + // Test that the correct number of pager links are found for exact phrase search. + $edit = array('keys' => '"love pizza"'); + $this->drupalPost('search/node', $edit, t('Search')); + $this->assertLinkByHref('page=1', 0, '2nd page link is found for exact phrase search.'); + $this->assertNoLinkByHref('page=2', '3rd page link is not found for exact phrase search.'); + } +} /** * Test integration searching comments.