Differential D16376

Add support for Differential Diffs to phid.lookup / PhabricatorObjectQuery
Needs ReviewPublic
Actions

Authored by dpkp on Aug 7 2016, 5:59 PM.

Tags

None

Referenced Files

	F19105711: D16376.diff
	Sat, Dec 6, 11:31 AM

	F18915181: D16376.diff
	Nov 9 2025, 7:32 AM

	F18906269: D16376.id.diff
	Nov 8 2025, 5:44 PM

	F18866633: D16376.id.diff
	Nov 3 2025, 6:22 PM

	F18860217: D16376.diff
	Nov 2 2025, 10:18 AM

	F18817966: D16376.diff
	Oct 21 2025, 7:56 PM

	F18803441: D16376.id39385.diff
	Oct 17 2025, 11:50 PM

	F18764531: D16376.id.diff
	Oct 7 2025, 6:33 AM

Subscribers

Details

Reviewers

joshuaspence
epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T11439: Retrieve Diff PHID via phid.lookup

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

Test Plan

Tested phid.lookup(names=["Diff 15"])

Diff Detail

Repository

Branch

phid_lookup_diff

Lint

Lint Passed

Unit

Tests Skipped

Build Status

Buildable 13262
Build 16999: arc lint + arc unit

Event Timeline

dpkp updated this revision to Diff 39385.Aug 7 2016, 5:59 PM

dpkp retitled this revision from to Add support for Differential Diffs to phid.lookup / PhabricatorObjectQuery.

dpkp updated this object.

dpkp edited the test plan for this revision. (Show Details)

dpkp added reviewers: epriestley, joshuaspence.

dpkp added a task: T11439: Retrieve Diff PHID via phid.lookup.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 7 2016, 5:59 PM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

Herald added a subscriber: Korvin. · View Herald Transcript

This is always super confusing because "Revisions" start with a "D" in their monogram, but a "diff" literally begins with the letter D while a revision does not.

I agree it's probably useful for this to exist but maybe we should come up with a better way of referring to a "diff" as opposed to a revision?

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

phid/

DifferentialDiffPHIDType.php

29 lines

Diff 39385

src/applications/differential/phid/DifferentialDiffPHIDType.php

Loading...