I don't believe that any subclass should override these methods.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- Restricted Diffusion Commit
rP934285a6d3d2: Mark some PhabricatorApplication methods as final
arc unit
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
I think it's plausible that a couple of these (like getApplicationURI() and isInstalled()) might eventually have some kind of use case, but we can add hooks or strip final if/when they do. Most of them seem unambiguously final.