Paths

Table of Contentst

Differential D19825

Expose "identifiers" as a query constraint for Commit search
ClosedPublic
Actions

Authored by epriestley on Nov 21 2018, 4:48 PM.

Tags

None

Referenced Files

	F18822313: D19825.diff
	Thu, Oct 23, 5:49 AM

	F18806734: D19825.id47362.diff
	Sat, Oct 18, 9:19 PM

	F18772292: D19825.id.diff
	Thu, Oct 9, 1:40 AM

	F18733851: D19825.id.diff
	Tue, Sep 30, 10:43 PM

	F18635053: D19825.diff
	Sep 17 2025, 12:34 AM

	F18625680: D19825.diff
	Sep 15 2025, 11:17 PM

	F18568945: D19825.id47362.diff
	Sep 10 2025, 12:31 AM

	F18568944: D19825.id47350.diff
	Sep 10 2025, 12:31 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13216: 2018 Week 45-47 Bonus Content

Commits

rP45e93c8f1d1a: Expose "identifiers" as a query constraint for Commit search

Summary

Ref T13216. See PHI984. The CommitSearchEngine (and, by extension, diffusion.commit.search) currently do not support identifier search, but this is a reasonable capability to provide.

Test Plan

Testing that a commit exists on master:

Screen Shot 2018-11-21 at 8.46.19 AM.png (1×2 px, 230 KB)

Same commit is not on stable:

Screen Shot 2018-11-21 at 8.46.36 AM.png (1×2 px, 159 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 21 2018, 4:48 PM

Harbormaster completed remote builds in B21198: Diff 47349.Nov 21 2018, 4:50 PM

epriestley requested review of this revision.Nov 21 2018, 4:50 PM

Slightly better identifiers, fewer documentation typos.

Harbormaster completed remote builds in B21199: Diff 47350.Nov 21 2018, 4:55 PM

amckinley accepted this revision.Nov 23 2018, 2:24 PM

This revision is now accepted and ready to land.Nov 23 2018, 2:24 PM

Closed by commit rP45e93c8f1d1a: Expose "identifiers" as a query constraint for Commit search (authored by epriestley). · Explain WhyNov 24 2018, 3:20 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

query/

PhabricatorCommitSearchEngine.php

13 lines

Diff 47362

View Options

src/applications/audit/query/PhabricatorCommitSearchEngine.php

Expose "identifiers" as a query constraint for Commit searchClosedPublicActions