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)
	Sun, Apr 14, 5:32 AM

	Unknown Object (File)
	Fri, Apr 12, 11:24 PM

	Unknown Object (File)
	Thu, Apr 11, 8:38 AM

	Unknown Object (File)
	Mon, Apr 8, 12:01 PM

	Unknown Object (File)
	Fri, Apr 5, 5:43 AM

	Unknown Object (File)
	Mon, Apr 1, 8:20 AM

	Unknown Object (File)
	Wed, Mar 20, 10:45 PM

	Unknown Object (File)
	Feb 18 2024, 12:07 PM

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...