Page MenuHomePhabricator

Add PhamePost to full text search
ClosedPublic

Authored by chad on Jun 14 2016, 6:23 PM.
Tags
None
Referenced Files
F14063981: D16116.diff
Mon, Nov 18, 9:22 PM
F14051009: D16116.diff
Thu, Nov 14, 9:55 PM
F14039093: D16116.diff
Mon, Nov 11, 3:44 AM
F14023430: D16116.diff
Thu, Nov 7, 2:02 AM
F14005782: D16116.id38780.diff
Sun, Oct 27, 7:41 PM
F14002520: D16116.id38779.diff
Fri, Oct 25, 7:40 PM
F13983269: D16116.id.diff
Sun, Oct 20, 5:12 AM
F13965784: D16116.diff
Oct 16 2024, 4:17 AM
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
Branch
PhamePost-Search (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12655
Build 16080: Run Core Tests
Build 16079: arc lint + arc unit

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.