Page MenuHomePhabricator

Add PhamePost to full text search
ClosedPublic

Authored by chad on Jun 14 2016, 6:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 4:15 PM
Unknown Object (File)
Fri, Apr 12, 1:13 PM
Unknown Object (File)
Fri, Apr 12, 1:13 PM
Unknown Object (File)
Mon, Apr 8, 11:28 PM
Unknown Object (File)
Thu, Apr 4, 4:06 PM
Unknown Object (File)
Tue, Apr 2, 8:24 PM
Unknown Object (File)
Sun, Mar 31, 4:14 PM
Unknown Object (File)
Wed, Mar 27, 4:59 PM
Subscribers

Details

Summary

Adds PhamePost object to fulltextsearch index. Some issue searching just "Open" though? Also "closed" objects search fine but don't display as disabled.

Test Plan

bin/search index --type POST

pasted_file (618×581 px, 69 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Add PhamePost to full text search.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.

To fix open/closed, you likely need to add some logic to PhabricatorPhamePostPHIDType so handles get marked as closed. You can look at, e.g., ManiphestTaskPHIDType for an example.

I'll see if I can repro the "open documents" thing, not immediately sure about that...

chad planned changes to this revision.Jun 14 2016, 6:34 PM

roger roger

Searching just "open" seems OK for me -- here's "open + closed", note result 1 is archived:

Screen Shot 2016-06-14 at 11.34.34 AM.png (525×470 px, 43 KB)

Here's just "open", no longer returns the closed result:

Screen Shot 2016-06-14 at 11.34.39 AM.png (509×467 px, 40 KB)

chad edited edge metadata.
  • Add closed UI
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 14 2016, 7:16 PM
This revision was automatically updated to reflect the committed changes.