Page MenuHomePhabricator
Differential D20803

Remove unused "icon" parameter from policy name rendering
ClosedPublic
Actions

Authored by epriestley on Sep 12 2019, 2:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 12:59 AM
Unknown Object (File)
Fri, Nov 29, 9:18 PM
Unknown Object (File)
Nov 26 2024, 8:39 AM
Unknown Object (File)
Nov 9 2024, 3:43 PM
Unknown Object (File)
Nov 2 2024, 10:31 PM
Unknown Object (File)
Oct 29 2024, 6:06 PM
Unknown Object (File)
Oct 22 2024, 10:25 PM
Unknown Object (File)
Oct 22 2024, 8:00 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13411: Improve "Custom Policy" behavior in policy dialogs
Commits
rPc9b0d107f079: Remove unused "icon" parameter from policy name rendering
Summary

Ref T13411. This pathway has an unused "icon" parameter with no callsites. Throw it away to ease refactoring.

Test Plan

Grepped for callsites, found none using this parameter.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 12 2019, 2:08 PM
Harbormaster completed remote builds in B23424: Diff 49599.Sep 12 2019, 2:10 PM
epriestley requested review of this revision.Sep 12 2019, 2:10 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 12 2019, 4:38 PM
Closed by commit rPc9b0d107f079: Remove unused "icon" parameter from policy name rendering (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPc9b0d107f079: Remove unused "icon" parameter from policy name rendering.

Revision Contents
Changeset List

PathSize
src/
applications/
policy/
controller/
4 lines
query/
5 lines
storage/
19 lines

Diff 49607

View Options

src/applications/policy/controller/PhabricatorPolicyExplainController.php

Loading...
View Options

src/applications/policy/query/PhabricatorPolicyQuery.php

Loading...
View Options

src/applications/policy/storage/PhabricatorPolicy.php

Loading...
Phabricator · Built by Phacility