Paths

Table of Contentst

Differential D18028

Modernize FeedSearchEngine a little bit
ClosedPublic
Actions

Authored by epriestley on May 26 2017, 5:11 PM.

Tags

None

Referenced Files

	F19046314: D18028.id.diff
	Thu, Nov 27, 10:04 AM

	F18780420: D18028.id.diff
	Oct 12 2025, 12:51 PM

	F18767730: D18028.diff
	Oct 8 2025, 12:03 AM

	F18766834: D18028.id43369.diff
	Oct 7 2025, 6:49 PM

	F18733620: D18028.id.diff
	Sep 30 2025, 10:38 PM

	F18657900: D18028.id43360.diff
	Sep 23 2025, 3:56 AM

	F18629972: D18028.id43369.diff
	Sep 16 2025, 10:24 AM

	F18619943: D18028.id43363.diff
	Sep 15 2025, 3:48 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12762: Feed queries should support date range constraints

Commits

rP6fed08a98e04: Modernize FeedSearchEngine a little bit

Summary

Ref T12762. This updates FeedSeachEngine to user modern construction.

I've tried to retain behavior exactly, although the "Include stories about projects I'm a member of" checkbox is now nonstandard/obsolete. We could likely fold that into "Include Projects" in a future change which does a backward compatibility break.

Test Plan

Queried feed without constraints.
Queried feed by user, project, "stuff I'm a member of", prebuilt "Tags" query.
Viewed user profile / project profile feeds.
Used function tokens (viewer(), etc).

Diff Detail

Repository

Branch

feed2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 17271
Build 23133: Run Core Tests
Build 23132: arc lint + arc unit

Event Timeline

epriestley created this revision.May 26 2017, 5:11 PM

Harbormaster completed remote builds in B17271: Diff 43360.May 26 2017, 5:12 PM

chad accepted this revision.May 26 2017, 5:34 PM

This revision is now accepted and ready to land.May 26 2017, 5:34 PM

Use a slightly less fancy project selector, since some of the advanced options do not work because this is not a real EdgeLogic query.

Harbormaster completed remote builds in B17274: Diff 43363.May 26 2017, 5:36 PM

Closed by commit rP6fed08a98e04: Modernize FeedSearchEngine a little bit (authored by epriestley, committed by epriestley). · Explain WhyMay 26 2017, 7:22 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T12851: The feed query '2HfUlTnoqZbw' causes an exception in PhabricatorSearchCheckboxesField.Jun 19 2017, 5:04 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

feed/

query/

PhabricatorFeedQuery.php

14 lines

PhabricatorFeedSearchEngine.php

93 lines

Diff 43360

src/applications/feed/query/PhabricatorFeedQuery.php

Loading...

src/applications/feed/query/PhabricatorFeedSearchEngine.php

Loading...