Paths

Table of Contentst

Differential D15295

Remove uncalled DiffusionRequest->getCallsign()
ClosedPublic
Actions

Authored by epriestley on Feb 18 2016, 1:15 AM.

Tags

None

Referenced Files

	F16416937: D15295.diff
	Mon, Jun 16, 7:47 AM

	Unknown Object (File)
	Mon, Jun 2, 5:09 AM

	Unknown Object (File)
	Thu, May 29, 4:31 PM

	Unknown Object (File)
	Tue, May 27, 10:51 AM

	Unknown Object (File)
	Tue, May 20, 6:26 PM

	Unknown Object (File)
	Apr 25 2025, 5:44 AM

	Unknown Object (File)
	Apr 25 2025, 1:21 AM

	Unknown Object (File)
	Apr 15 2025, 5:50 PM

Subscribers

None

Details

Reviewers

avivey
chad

Maniphest Tasks

T4245: Make repository URIs slightly prettier

Commits

Restricted Diffusion Commit
rP93d7b0122252: Remove uncalled DiffusionRequest->getCallsign()

Summary

Ref T4245. This has no callers.

Test Plan

Ran git grep -i 'getCallsign(' and visually verified that no callers could reasonably be DiffusionRequest objects (there are only 23 remaining sites, and about half are $this->... in PhabricatorRepository.
Browsed around directory/file/branch/content/diff/etc pages in Diffusion.

Diff Detail

Repository

Branch

call3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 10742
Build 13215: Run Core Tests
Build 13214: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36899.Feb 18 2016, 1:15 AM

epriestley retitled this revision from to Remove uncalled DiffusionRequest->getCallsign().

epriestley updated this object.

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

epriestley added reviewers: chad, avivey.

epriestley added a task: T4245: Make repository URIs slightly prettier.

avivey accepted this revision.Feb 18 2016, 1:17 AM

avivey edited edge metadata.

This revision is now accepted and ready to land.Feb 18 2016, 1:17 AM

Closed by commit rP93d7b0122252: Remove uncalled DiffusionRequest->getCallsign() (authored by epriestley, committed by epriestley). · Explain WhyFeb 18 2016, 1:17 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

request/

DiffusionRequest.php

4 lines

Diff 36899

src/applications/diffusion/request/DiffusionRequest.php

Loading...