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

	Unknown Object (File)
	Wed, Feb 26, 12:02 PM

	Unknown Object (File)
	Wed, Feb 26, 12:01 PM

	Unknown Object (File)
	Sun, Feb 23, 10:00 PM

	Unknown Object (File)
	Sat, Feb 15, 3:11 PM

	Unknown Object (File)
	Fri, Feb 14, 4:28 PM

	Unknown Object (File)
	Mon, Feb 10, 3:33 AM

	Unknown Object (File)
	Sat, Feb 8, 4:22 AM

	Unknown Object (File)
	Sat, Feb 8, 4:22 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

Branch

fulluri1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5826
Build 5846: [Placeholder Plan] Wait for 30 Seconds

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 30635

src/applications/differential/customfield/DifferentialRevisionIDField.php

Loading...