Page MenuHomePhabricator

Make PhameBlog full text searchable
ClosedPublic

Authored by chad on Jun 14 2016, 7:58 PM.
Tags
None
Referenced Files
F13254769: D16119.diff
Sat, May 25, 3:44 AM
F13239129: D16119.id.diff
Wed, May 22, 12:24 AM
F13232522: D16119.id.diff
Tue, May 21, 1:20 AM
F13212539: D16119.diff
Fri, May 17, 6:51 AM
F13208870: D16119.id38785.diff
Thu, May 16, 8:13 PM
F13196151: D16119.diff
Sun, May 12, 11:00 PM
F13183766: D16119.id38784.diff
Fri, May 10, 12:13 PM
F13180398: D16119.diff
Wed, May 8, 11:43 PM
Subscribers

Details

Summary

Ref T9897, makes blogs searchable

Test Plan

Make a blog, index it, search for it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Make PhameBlog full text searchable.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 14 2016, 8:11 PM
This revision was automatically updated to reflect the committed changes.

Oh, these might also need to implement supportsSearch() or whatever in the Editor classes. That should really become automatic if the object implements FulltextInterface, but currently isn't.

You can test by editing a blog/post to contain a unique word, then searching for it. If you fail to find it, check if supportsSearch() is implemented to return true;.

(That is, as long as the daemons are running, you shouldn't normally need to manually rebuild indexes for search results to reflect changes.)