Page MenuHomePhabricator
Differential D14217

Use getPhobjectClassConstant() to access class constants
ClosedPublic
Actions

Authored by epriestley on Oct 1 2015, 8:27 PM.
Tags
None
Referenced Files
F15474827: D14217.id34330.diff
Sun, Apr 6, 12:54 PM
F15473868: D14217.diff
Sun, Apr 6, 2:32 AM
F15471822: D14217.diff
Sat, Apr 5, 8:14 AM
F15462360: D14217.id34330.diff
Tue, Apr 1, 2:25 PM
F15458159: D14217.id34330.diff
Sun, Mar 30, 9:22 PM
F15445102: D14217.id34336.diff
Thu, Mar 27, 12:26 PM
F15437550: D14217.id.diff
Tue, Mar 25, 8:17 PM
F15436145: D14217.id34336.diff
Mar 25 2025, 12:15 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9494: Standardize GarbageCollector TTLs
Commits
Restricted Diffusion Commit
rPe431ab218923: Use getPhobjectClassConstant() to access class constants
Summary

Ref T9494. Depends on D14216. Remove 10 copies of this code.

Test Plan

Ran arc unit --everything, browsed Config > Modules, clicked around Herald / etc.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 34330.Oct 1 2015, 8:27 PM
epriestley retitled this revision from to Use getPhobjectClassConstant() to access class constants.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T9494: Standardize GarbageCollector TTLs.
epriestley added a parent revision: D14216: Add getPhobjectClassConstant() to Phobject.
chad accepted this revision.Oct 1 2015, 8:32 PM
chad updated this object.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 1 2015, 8:33 PM
Closed by commit rPe431ab218923: Use getPhobjectClassConstant() to access class constants (authored by epriestley, committed by epriestley). · Explain WhyOct 1 2015, 11:56 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
drydock/
logtype/
30 lines
harbormaster/
artifact/
30 lines
stepgroup/
14 lines
herald/
action/
30 lines
14 lines
field/
28 lines
14 lines
phid/
type/
12 lines
policy/
capability/
24 lines
infrastructure/
edges/
type/
12 lines

Diff 34336

View Options

src/applications/drydock/logtype/DrydockLogType.php

Loading...
View Options

src/applications/harbormaster/artifact/HarbormasterArtifact.php

Loading...
View Options

src/applications/harbormaster/stepgroup/HarbormasterBuildStepGroup.php

Loading...
View Options

src/applications/herald/action/HeraldAction.php

Loading...
View Options

src/applications/herald/action/HeraldActionGroup.php

Loading...
View Options

src/applications/herald/field/HeraldField.php

Loading...
View Options

src/applications/herald/field/HeraldFieldGroup.php

Loading...
View Options

src/applications/phid/type/PhabricatorPHIDType.php

Loading...
View Options

src/applications/policy/capability/PhabricatorPolicyCapability.php

Loading...
View Options

src/infrastructure/edges/type/PhabricatorEdgeType.php

Loading...
Phabricator · Built by Phacility