revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-20 20:40:56 +01:00
Code Issues Releases Wiki Activity

Only index documentable atoms for search

Browse source 
Summary: Ref T7458. Only index documentable atoms in the search index. In particular, this prevents files and methods from being returned in search results (clicking on these search results doesn't actually work anyway).

Test Plan: Actually, to get this to work I had to destroy the search index and recreate it... is this expected?

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Maniphest Tasks: T7458

Differential Revision: https://secure.phabricator.com/D13298
This commit is contained in:
Joshua Spence 2015-06-16 06:51:51 +10:00
parent a1561a4d3a
commit 600a3e3b7c
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/diviner/search/DivinerAtomSearchIndexer.php
View file

@ -10,6 +10,10 @@ final class DivinerAtomSearchIndexer extends PhabricatorSearchDocumentIndexer {
$atom = $this->loadDocumentByPHID($phid); $atom = $this->loadDocumentByPHID($phid);
$book = $atom->getBook(); $book = $atom->getBook();
if (!$atom->getIsDocumentable()) {
return null;
}
$doc = $this->newDocument($phid) $doc = $this->newDocument($phid)
->setDocumentTitle($atom->getTitle()) ->setDocumentTitle($atom->getTitle())
->setDocumentCreated($book->getDateCreated()) ->setDocumentCreated($book->getDateCreated())