Paths

Table of Contentst

Differential D12749

Allow Phabricator to parse bare revision IDs from "Differential Revision:" fields
ClosedPublic
Actions

Authored by epriestley on May 7 2015, 1:01 PM.

Tags

None

Referenced Files

	F15541768: D12749.diff
	Fri, Apr 25, 1:30 PM

	F15505463: D12749.diff
	Tue, Apr 15, 3:11 AM

	F15502849: D12749.id30635.diff
	Mon, Apr 14, 4:51 AM

	F15445762: D12749.id30655.diff
	Thu, Mar 27, 3:52 PM

	F15439474: D12749.id.diff
	Mar 26 2025, 7:42 AM

	F15438021: D12749.diff
	Mar 25 2025, 10:50 PM

	F15423089: D12749.id.diff
	Mar 22 2025, 12:49 PM

	F15391682: D12749.diff
	Mar 15 2025, 11:03 AM

Subscribers

Tokens

"Orange Medal" token, awarded by cburroughs.

Details

Reviewers

Maniphest Tasks

T8087: Let Phabricator parse either "Dxxx" (no URI) or "https://secure.phabricator.com/Dxxx" (full URI) in Differential Revisions

Commits

Restricted Diffusion Commit
rP88b1ad7e92e5: Allow Phabricator to parse bare revision IDs from "Differential Revision:"…

Summary

Fixes T8087. Depends on D12748.

Test Plan

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 30635.May 7 2015, 1:01 PM

epriestley retitled this revision from to Allow Phabricator to parse bare revision IDs from "Differential Revision:" fields.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T8087: Let Phabricator parse either "Dxxx" (no URI) or "https://secure.phabricator.com/Dxxx" (full URI) in Differential Revisions.

epriestley added a parent revision: D12748: Allow Arcanist to parse bare revision IDs from "Differential Revisions:" field.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 7 2015, 1:01 PM

epriestley added a subscriber: cburroughs.May 7 2015, 1:03 PM

epriestley added inline comments.

src/applications/differential/customfield/DifferentialRevisionIDField.php
50–51	@cburroughs, to get the short `D123` behavior (instead of full URIs) in advance of T6030: Wait for D12748 + D12749 (this change). Get everyone to upgrade "arc" past D12748. Make this change: - return PhabricatorEnv::getProductionURI('/D'.$id); + return 'D'.$id;

cburroughs awarded a token.May 7 2015, 4:24 PM

btrahan accepted this revision.May 7 2015, 5:44 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 7 2015, 5:44 PM

Closed by commit rP88b1ad7e92e5: Allow Phabricator to parse bare revision IDs from "Differential Revision:"… (authored by epriestley, committed by epriestley). · Explain WhyMay 7 2015, 6:10 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

customfield/

DifferentialRevisionIDField.php

8 lines

Diff 30655

src/applications/differential/customfield/DifferentialRevisionIDField.php

Loading...