Page MenuHomePhabricator

Mark `PhabricatorPHIDType::getPHIDTypeApplicationClass()` as abstract
ClosedPublic

Authored by joshuaspence on Sep 6 2015, 3:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 8:17 AM
Unknown Object (File)
Thu, Apr 18, 1:42 AM
Unknown Object (File)
Thu, Apr 11, 10:09 AM
Unknown Object (File)
Tue, Apr 9, 10:40 AM
Unknown Object (File)
Wed, Apr 3, 11:40 AM
Unknown Object (File)
Wed, Apr 3, 11:40 AM
Unknown Object (File)
Wed, Apr 3, 11:40 AM
Unknown Object (File)
Wed, Apr 3, 11:40 AM
Subscribers

Details

Summary

Fixes T9625. As explained in a TODO comment, seems reasonable enough.

Test Plan

Unit tests.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Mark `PhabricatorPHIDType::getPHIDTypeApplicationClass()` as abstract.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

(I vaguely wonder if we should have a PhabricatorWhateverApplication::APPCONST sort of constant, mostly to get better static analyzability, but that's easy to add later.)

This revision is now accepted and ready to land.Sep 6 2015, 2:58 PM
This revision was automatically updated to reflect the committed changes.