Search changes.

Fixes #1376: Only fetching ft_min_word_length if the query != false.
Using IN BOOLEAN MODE for metadata search (comments, tags) to avoid a fulltext index on the metastrings table.  Slower for search, faster for site.
Simplified and modularized logic for pulling out relevant substrings and highlighting.
Corrected the elipse oddities in relevancy substring concatenation.
Added pagination on non-homepages.
Added missing language strings.
Updated and standardized comment results listings.
Repeat query in searchbar.
Dealing with comments on unavailable entities better (though not well).
Increased default word context to 30 characters.
Decreased default context max length to 300 characters.
Promise to start making atomic commits real soon now.



git-svn-id: http://code.elgg.org/elgg/trunk@3684 36083f99-b078-4883-b0ff-0f9b5a30f544

1 files changed, 57 insertions, 13 deletions

diff --git a/mod/search/views/default/search/comments/listing.php b/mod/search/views/default/search/comments/listing.php
index 58353a110..13f368f2b 100644
--- a/mod/search/views/default/search/comments/listing.php
+++ b/mod/search/views/default/search/comments/listing.php
@@ -7,16 +7,12 @@
  * @author Curverider Ltd
  * @link http://elgg.org/
  */
-?>
 
-<div class="search_listing">
-<?php
 if (!is_array($vars['entities']) || !count($vars['entities'])) {
 	return FALSE;
 }
 
 $title_str = elgg_echo('comments');
-$body = elgg_view_title($title_str);
 
 $query = htmlspecialchars(http_build_query(
 	array(
@@ -30,9 +26,33 @@ $query = htmlspecialchars(http_build_query(
 ));
 
 $url = "{$vars['url']}pg/search?$query";
-$more = "<a href=\"$url\">+$count more $title_str</a>";
 
-echo elgg_view('page_elements/contentwrapper', array('body' => $body));
+// get pagination
+if (array_key_exists('pagination', $vars) && $vars['pagination']) {
+	$nav .= elgg_view('navigation/pagination',array(
+		'baseurl' => $url,
+		'offset' => $vars['params']['offset'],
+		'count' => $vars['count'],
+		'limit' => $vars['params']['limit'],
+	));
+} else {
+	$nav = '';
+}
+
+// get more links
+$more_check = $vars['count'] - ($vars['params']['offset'] + $vars['params']['limit']);
+$more = ($more_check > 0) ? $more_check : 0;
+
+if ($more) {
+	$title_key = ($more == 1) ? 'comment' : 'comments';
+	$more_str = sprintf(elgg_echo('search:more'), $vars['count'], elgg_echo($title_key));
+	$more_link = "<a href=\"$url\">$more_str</a>";
+} else {
+	$more_link = '';
+}
+
+echo $nav;
+$body = elgg_view_title($title_str);
 
 foreach ($vars['entities'] as $entity) {
 	if ($owner = $entity->getOwnerEntity()) {
@@ -41,22 +61,46 @@ foreach ($vars['entities'] as $entity) {
 	} else {
 		$icon = '';
 	}
-	$title = "Comment on " . elgg_echo('item:' . $entity->getType() . ':' . $entity->getSubtype());
+
+	// @todo Sometimes we find comments on entities we can't display...
+	if ($entity->getVolatileData('search_unavailable_entity')) {
+		$title = sprintf(elgg_echo('search:comment_on'), elgg_echo('search:unavailable_entity'));
+		// keep anchor for formatting.
+		$title = "<a>$title</a>";
+	} else {
+		if ($entity->getType() == 'object') {
+			$title = $entity->title;
+		} else {
+			$title = $entity->name;
+		}
+
+		if (!$title) {
+			$title = elgg_echo('item:' . $entity->getType() . ':' . $entity->getSubtype());
+		}
+
+		if (!$title) {
+			$title = elgg_echo('item:' . $entity->getType());
+		}
+
+		$title = sprintf(elgg_echo('search:comment_on'), $title);
+		$url = $entity->getURL() . '#annotation-' . $entity->getVolatileData('search_match_annotation_id');
+		$title = "<a href=\"$url\">$title</a>";
+	}
+
 	$description = $entity->getVolatileData('search_matched_comment');
-	$url = $entity->getURL();
-	$title = "<a href=\"$url\">$title</a>";
 	$tc = $entity->getVolatileData('search_matched_comment_time_created');;
 	$time = friendly_time($tc);
 
-	echo <<<___END
+	$body .= <<<___END
 <span class="searchListing">
 	<h3 class="searchTitle">$title</h3>
 	<span class="searchDetails">
 		<span class="searchDescription">$description</span><br />
-		$icon $time - $more</a>
+		$icon $time - $more_link</a>
 	</span>
 </span>
 ___END;
 }
-?>
-</div>
+
+$body .= $nav;
+echo elgg_view('page_elements/contentwrapper', array('body' => $body));