Reports: Remove unused variable
ClosedPublic
Actions

Authored by aklapper on Tue, Apr 30, 16:22.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, May 10, 15:27

	Unknown Object (File)
	Fri, May 10, 12:27

	Unknown Object (File)
	Fri, May 10, 00:35

	Unknown Object (File)
	Tue, May 7, 07:28

	Unknown Object (File)
	Tue, May 7, 03:01

	Unknown Object (File)
	Sun, May 5, 23:32

	Unknown Object (File)
	Sun, May 5, 23:32

	Unknown Object (File)
	Sun, May 5, 23:32

Subscribers

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rP74e28bc2a7b0: Reports: Remove unused variable

Summary

This variable feels unneeded in this function. It may have a better and more fulfilling life after finding a new function.

Test Plan

Read slowly and aloud all code in the function.

Diff Detail

Repository

rP Phorge

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Tue, Apr 30, 16:22

Owners added a reviewer: O1: Blessed Committers.Tue, Apr 30, 16:22

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptTue, Apr 30, 16:22

aklapper requested review of this revision.Tue, Apr 30, 16:22

Harbormaster completed remote builds in B1209: Diff 1914.Tue, Apr 30, 16:22

Phorge saves CPU cycles

This revision is now accepted and ready to land.Tue, Apr 30, 18:57

Closed by commit rP74e28bc2a7b0: Reports: Remove unused variable. · Explain WhyWed, May 1, 09:44

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP74e28bc2a7b0: Reports: Remove unused variable.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

project/

controller/

PhabricatorProjectReportsController.php

5 lines

O1: Blessed Committers

Diff 1916

View Options

src/applications/project/controller/PhabricatorProjectReportsController.php

Reports: Remove unused variableClosedPublicActions