Differential D20208

Support an "authorPHIDs" constraint for "transaction.search"
ClosedPublic
Actions

Authored by epriestley on Feb 25 2019, 1:02 PM.

Tags

None

Referenced Files

	F19802327: D20208.id.diff
	Sun, Mar 1, 3:55 PM

	F19678316: D20208.id48253.diff
	Sun, Feb 8, 4:05 PM

	F19678281: D20208.id48253.diff
	Sun, Feb 8, 4:02 PM

	F19553437: D20208.id48249.diff
	Jan 29 2026, 11:01 AM

	F19508556: D20208.diff
	Jan 10 2026, 6:42 AM

	F19095533: D20208.id.diff
	Dec 4 2025, 5:30 AM

	F19090829: D20208.diff
	Dec 3 2025, 12:42 PM

	F18860798: D20208.diff
	Nov 2 2025, 12:44 PM

View All Files

Subscribers

None

Tokens

"Mountain of Wealth" token, awarded by 20after4.

Details

Reviewers

amckinley

Maniphest Tasks

T13255: transaction.search should accept user PHIDs as constraints

Commits

rP767afd1780fd: Support an "authorPHIDs" constraint for "transaction.search"

Summary

Ref T13255. The "transaction.search" API method currently does not support author constraints, but this is a reasonable thing to support.

Test Plan

Queried transactions by author, hit the error cases.

Diff Detail

Repository

rP Phabricator

Branch

xaction1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 22110
Build 30205: Run Core Tests
Build 30204: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 25 2019, 1:02 PM

Harbormaster completed remote builds in B22110: Diff 48249.Feb 25 2019, 1:04 PM

epriestley requested review of this revision.Feb 25 2019, 1:04 PM

amckinley accepted this revision.Feb 25 2019, 1:27 PM

This revision is now accepted and ready to land.Feb 25 2019, 1:27 PM

epriestley added a child revision: D20209: Enrich the "change project tags" transaction in "transaction.search".Feb 25 2019, 1:35 PM

20after4 awarded a token.Feb 25 2019, 1:52 PM

Closed by commit rP767afd1780fd: Support an "authorPHIDs" constraint for "transaction.search" (authored by epriestley). · Explain WhyFeb 25 2019, 2:33 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T13255: transaction.search should accept user PHIDs as constraints.Feb 25 2019, 7:08 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

transactions/

conduit/

TransactionSearchConduitAPIMethod.php

59 lines

Diff 48249

View Options

src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php

Support an "authorPHIDs" constraint for "transaction.search"ClosedPublicActions