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

	Unknown Object (File)
	Tue, Nov 12, 11:44 AM

	Unknown Object (File)
	Wed, Nov 6, 8:14 PM

	Unknown Object (File)
	Tue, Nov 5, 11:52 PM

	Unknown Object (File)
	Tue, Nov 5, 11:52 PM

	Unknown Object (File)
	Tue, Nov 5, 11:52 PM

	Unknown Object (File)
	Tue, Nov 5, 11:52 PM

	Unknown Object (File)
	Mon, Nov 4, 10:26 AM

	Unknown Object (File)
	Tue, Oct 29, 11:59 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 17274
Build 23139: Run Core Tests
Build 23138: 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

96 lines

Diff 43363

src/applications/feed/query/PhabricatorFeedQuery.php

Loading...

src/applications/feed/query/PhabricatorFeedSearchEngine.php

Loading...