tonyair
/
silverstripe-fulltextsearch
mirror of https://github.com/silverstripe/silverstripe-fulltextsearch
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

BUG Remove non-displayable chars from Solr highlight results

This commit is contained in:
Ingo Schommer 2013-03-11 10:50:14 +01:00
parent 919716a0ce
commit a7629c8bb2
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
code/solr/SolrIndex.php
View File

@ -312,7 +312,7 @@ abstract class SolrIndex extends SearchIndex {
*/ */
public function search(SearchQuery $query, $offset = -1, $limit = -1, $params = array()) { public function search(SearchQuery $query, $offset = -1, $limit = -1, $params = array()) {
$service = $this->getService(); $service = $this->getService();
SearchVariant::with(count($query->classes) == 1 ? $query->classes[0]['class'] : null)->call('alterQuery', $query, $this); SearchVariant::with(count($query->classes) == 1 ? $query->classes[0]['class'] : null)->call('alterQuery', $query, $this);
$q = array(); $q = array();
@ -440,6 +440,9 @@ abstract class SolrIndex extends SearchIndex {
$combinedHighlights = array_merge($combinedHighlights, $highlights); $combinedHighlights = array_merge($combinedHighlights, $highlights);
} }
$result->Excerpt = DBField::create_field('HTMLText', implode(' ... ', $combinedHighlights)); $result->Excerpt = DBField::create_field('HTMLText', implode(' ... ', $combinedHighlights));
// Remove entity-encoded U+FFFD REPLACEMENT CHARACTER.
// It signifies non-displayable characters, and shows up as such itself in browsers (questionmark icon)
$result->Excerpt = str_replace('�', '', $result->Excerpt);
} }
} }
} }
@ -447,7 +450,7 @@ abstract class SolrIndex extends SearchIndex {
} else { } else {
$numFound = 0; $numFound = 0;
} }
$ret = array(); $ret = array();
$ret['Matches'] = new PaginatedList($results); $ret['Matches'] = new PaginatedList($results);
$ret['Matches']->setLimitItems(false); $ret['Matches']->setLimitItems(false);