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

	F14745049: D15295.diff
	Tue, Jan 21, 9:17 AM

	Unknown Object (File)
	Tue, Jan 14, 10:28 PM

	Unknown Object (File)
	Thu, Jan 2, 1:59 PM

	Unknown Object (File)
	Thu, Jan 2, 6:54 AM

	Unknown Object (File)
	Tue, Dec 31, 12:23 AM

	Unknown Object (File)
	Sun, Dec 29, 5:49 PM

	Unknown Object (File)
	Dec 22 2024, 3:16 PM

	Unknown Object (File)
	Dec 22 2024, 10:08 AM

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...