diff options
| author | Brett Profitt <brett.profitt@gmail.com> | 2012-02-25 12:58:21 -0800 |
|---|---|---|
| committer | Brett Profitt <brett.profitt@gmail.com> | 2012-02-25 12:58:21 -0800 |
| commit | 6c94d823bbea21a5ab5672a3d7780ad598e1c665 (patch) | |
| tree | 52759cb28c782fab41dec73f806091dd350693ce /mod/search | |
| parent | 6c7ff96f073dc35a9e43e80e80121f2b0463b06d (diff) | |
| download | elgg-6c94d823bbea21a5ab5672a3d7780ad598e1c665.tar.gz elgg-6c94d823bbea21a5ab5672a3d7780ad598e1c665.tar.bz2 | |
Fixes #3550. Merged slightly modified pull request 166 from sembrestels.
Diffstat (limited to 'mod/search')
| -rw-r--r-- | mod/search/pages/search/index.php | 6 | ||||
| -rw-r--r-- | mod/search/search_hooks.php | 2 | ||||
| -rw-r--r-- | mod/search/start.php | 7 |
3 files changed, 11 insertions, 4 deletions
diff --git a/mod/search/pages/search/index.php b/mod/search/pages/search/index.php index 91817096b..fcd95c43e 100644 --- a/mod/search/pages/search/index.php +++ b/mod/search/pages/search/index.php @@ -257,7 +257,11 @@ if ($search_type != 'entities' || $search_type == 'all') { } // highlight search terms -$searched_words = search_remove_ignored_words($display_query, 'array'); +if ($search_type == 'tags') { + $searched_words = array($display_query); +} else { + $searched_words = search_remove_ignored_words($display_query, 'array'); +} $highlighted_query = search_highlight_words($searched_words, $display_query); $body = elgg_view_title(elgg_echo('search:results', array("\"$highlighted_query\""))); diff --git a/mod/search/search_hooks.php b/mod/search/search_hooks.php index ab000f6f6..2143a0d24 100644 --- a/mod/search/search_hooks.php +++ b/mod/search/search_hooks.php @@ -284,7 +284,7 @@ function search_tags_hook($hook, $type, $value, $params) { } $tags_str = implode('. ', $matched_tags_strs); - $tags_str = search_get_highlighted_relevant_substrings($tags_str, $params['query']); + $tags_str = search_get_highlighted_relevant_substrings($tags_str, $params['query'], 30, 300, true); $entity->setVolatileData('search_matched_title', $title_str); $entity->setVolatileData('search_matched_description', $desc_str); diff --git a/mod/search/start.php b/mod/search/start.php index bb8531e9c..d2d7ed3c2 100644 --- a/mod/search/start.php +++ b/mod/search/start.php @@ -83,15 +83,18 @@ function search_page_handler($page) { * @param string $query * @param int $min_match_context = 30 * @param int $max_length = 300 + * @param bool $tag_match Search is for tags. Don't ignore words. * @return string */ -function search_get_highlighted_relevant_substrings($haystack, $query, $min_match_context = 30, $max_length = 300) { +function search_get_highlighted_relevant_substrings($haystack, $query, $min_match_context = 30, $max_length = 300, $tag_match = false) { $haystack = strip_tags($haystack); $haystack_length = elgg_strlen($haystack); $haystack_lc = elgg_strtolower($haystack); - $words = search_remove_ignored_words($query, 'array'); + if (!$tag_match) { + $words = search_remove_ignored_words($query, 'array'); + } // if haystack < $max_length return the entire haystack w/formatting immediately if ($haystack_length <= $max_length) { |