X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/b80184cd93d502ab2b4c0f21c1139842ca946057..refs/pull/255/head:/app/Entity.php diff --git a/app/Entity.php b/app/Entity.php index 7347ddcd6..186059f00 100644 --- a/app/Entity.php +++ b/app/Entity.php @@ -60,7 +60,7 @@ class Entity extends Ownable */ public function tags() { - return $this->morphMany(Tag::class, 'entity'); + return $this->morphMany(Tag::class, 'entity')->orderBy('order', 'asc'); } /** @@ -157,24 +157,34 @@ class Entity extends Ownable * @param string[] array $wheres * @return mixed */ - public static function fullTextSearchQuery($fieldsToSearch, $terms, $wheres = []) + public function fullTextSearchQuery($fieldsToSearch, $terms, $wheres = []) { $exactTerms = []; + $fuzzyTerms = []; + $search = static::newQuery(); + foreach ($terms as $key => $term) { $term = htmlentities($term, ENT_QUOTES); $term = preg_replace('/[+\-><\(\)~*\"@]+/', ' ', $term); - if (preg_match('/\s/', $term)) { + if (preg_match('/".*?"/', $term) || is_numeric($term)) { + $term = str_replace('"', '', $term); $exactTerms[] = '%' . $term . '%'; - $term = '"' . $term . '"'; } else { $term = '' . $term . '*'; + if ($term !== '*') $fuzzyTerms[] = $term; } - if ($term !== '*') $terms[$key] = $term; } - $termString = implode(' ', $terms); - $fields = implode(',', $fieldsToSearch); - $search = static::selectRaw('*, MATCH(name) AGAINST(? IN BOOLEAN MODE) AS title_relevance', [$termString]); - $search = $search->whereRaw('MATCH(' . $fields . ') AGAINST(? IN BOOLEAN MODE)', [$termString]); + + $isFuzzy = count($exactTerms) === 0 && count($fuzzyTerms) > 0; + + + // Perform fulltext search if relevant terms exist. + if ($isFuzzy) { + $termString = implode(' ', $fuzzyTerms); + $fields = implode(',', $fieldsToSearch); + $search = $search->selectRaw('*, MATCH(name) AGAINST(? IN BOOLEAN MODE) AS title_relevance', [$termString]); + $search = $search->whereRaw('MATCH(' . $fields . ') AGAINST(? IN BOOLEAN MODE)', [$termString]); + } // Ensure at least one exact term matches if in search if (count($exactTerms) > 0) { @@ -187,18 +197,21 @@ class Entity extends Ownable }); } + $orderBy = $isFuzzy ? 'title_relevance' : 'updated_at'; + // Add additional where terms foreach ($wheres as $whereTerm) { $search->where($whereTerm[0], $whereTerm[1], $whereTerm[2]); } + // Load in relations - if (static::isA('page')) { + if ($this->isA('page')) { $search = $search->with('book', 'chapter', 'createdBy', 'updatedBy'); - } else if (static::isA('chapter')) { + } else if ($this->isA('chapter')) { $search = $search->with('book'); } - return $search->orderBy('title_relevance', 'desc'); + return $search->orderBy($orderBy, 'desc'); } - + }