Page MenuHomePhabricator

Rename `PHIDType` classes
ClosedPublic

Authored by joshuaspence on Jul 19 2014, 11:53 AM.
Tags
None
Referenced Files
F14477698: D9986.diff
Sat, Dec 28, 3:10 PM
F14477377: D9986.id24064.diff
Sat, Dec 28, 1:55 PM
Unknown Object (File)
Fri, Dec 20, 5:59 PM
Unknown Object (File)
Wed, Dec 11, 10:47 AM
Unknown Object (File)
Mon, Dec 9, 12:18 AM
Unknown Object (File)
Sun, Dec 8, 9:03 AM
Unknown Object (File)
Fri, Dec 6, 11:30 AM
Unknown Object (File)
Tue, Dec 3, 11:26 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T5655: Standardize naming conventions
Commits
Restricted Diffusion Commit
rP97a8700e4589: Rename `PHIDType` classes
Summary

Ref T5655. Rename PhabricatorPHIDType subclasses for clarity (see discussion in D9839). I'm not too keen on some of the resulting class names, so feel free to suggest alternatives.

Test Plan

Ran unit tests.

Diff Detail

Repository
rP Phabricator
Branch
phidtype
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1855
Build 1856: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Rename `PHIDType` classes.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Some of these names are a bit cumbersome, but I think that's just the price we pay for consistency.

This revision is now accepted and ready to land.Jul 21 2014, 2:07 PM

Although less consistent, I somewhat think that PhabricatorApplicationApplicationPHIDType would be better as PhabricatorApplicationPHIDType, for example.

bizrad6 added inline comments.
resources/sql/patches/20131020.pxactionmig.php
51

I think this one was missed. Should be:

PhabricatorProjectProjectPHIDType::TYPECONST);