Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F13978728
D13265.id32114.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
4 KB
Referenced Files
None
Subscribers
None
D13265.id32114.diff
View Options
diff --git a/src/applications/base/PhabricatorApplication.php b/src/applications/base/PhabricatorApplication.php
--- a/src/applications/base/PhabricatorApplication.php
+++ b/src/applications/base/PhabricatorApplication.php
@@ -17,7 +17,7 @@
const GROUP_ADMIN = 'admin';
const GROUP_DEVELOPER = 'developer';
- public static function getApplicationGroups() {
+ final public static function getApplicationGroups() {
return array(
self::GROUP_CORE => pht('Core Applications'),
self::GROUP_UTILITIES => pht('Utilities'),
@@ -35,7 +35,7 @@
return pht('%s Application', $this->getName());
}
- public function isInstalled() {
+ final public function isInstalled() {
if (!$this->canUninstall()) {
return true;
}
@@ -133,7 +133,7 @@
return true;
}
- public function getPHID() {
+ final public function getPHID() {
return 'PHID-APPS-'.get_class($this);
}
@@ -145,7 +145,7 @@
return null;
}
- public function getApplicationURI($path = '') {
+ final public function getApplicationURI($path = '') {
return $this->getBaseURI().ltrim($path, '/');
}
@@ -169,7 +169,7 @@
return null;
}
- public function getHelpMenuItems(PhabricatorUser $viewer) {
+ final public function getHelpMenuItems(PhabricatorUser $viewer) {
$items = array();
$articles = $this->getHelpDocumentationArticles($viewer);
@@ -249,7 +249,7 @@
return false;
}
- protected function getInboundEmailSupportLink() {
+ final protected function getInboundEmailSupportLink() {
return PhabricatorEnv::getDocLink('Configuring Inbound Email');
}
@@ -286,7 +286,7 @@
* @return string
* @task ui
*/
- public static function formatStatusCount(
+ final public static function formatStatusCount(
$count,
$limit_string = '%s',
$base_string = '%d') {
@@ -359,7 +359,7 @@
/* -( Application Management )--------------------------------------------- */
- public static function getByClass($class_name) {
+ final public static function getByClass($class_name) {
$selected = null;
$applications = self::getAllApplications();
@@ -377,7 +377,7 @@
return $selected;
}
- public static function getAllApplications() {
+ final public static function getAllApplications() {
static $applications;
if ($applications === null) {
@@ -401,7 +401,7 @@
return $applications;
}
- public static function getAllInstalledApplications() {
+ final public static function getAllInstalledApplications() {
$all_applications = self::getAllApplications();
$apps = array();
foreach ($all_applications as $app) {
@@ -426,7 +426,7 @@
* @return bool True if the class is installed.
* @task meta
*/
- public static function isClassInstalled($class) {
+ final public static function isClassInstalled($class) {
return self::getByClass($class)->isInstalled();
}
@@ -443,7 +443,7 @@
* @return bool True if the class is installed for the viewer.
* @task meta
*/
- public static function isClassInstalledForViewer(
+ final public static function isClassInstalledForViewer(
$class,
PhabricatorUser $viewer) {
@@ -502,7 +502,7 @@
return array();
}
- private function getCustomPolicySetting($capability) {
+ final private function getCustomPolicySetting($capability) {
if (!$this->isCapabilityEditable($capability)) {
return null;
}
@@ -528,7 +528,7 @@
}
- private function getCustomCapabilitySpecification($capability) {
+ final private function getCustomCapabilitySpecification($capability) {
$custom = $this->getCustomCapabilities();
if (!isset($custom[$capability])) {
throw new Exception(pht("Unknown capability '%s'!", $capability));
@@ -536,7 +536,7 @@
return $custom[$capability];
}
- public function getCapabilityLabel($capability) {
+ final public function getCapabilityLabel($capability) {
switch ($capability) {
case PhabricatorPolicyCapability::CAN_VIEW:
return pht('Can Use Application');
@@ -552,7 +552,7 @@
return null;
}
- public function isCapabilityEditable($capability) {
+ final public function isCapabilityEditable($capability) {
switch ($capability) {
case PhabricatorPolicyCapability::CAN_VIEW:
return $this->canUninstall();
@@ -564,7 +564,7 @@
}
}
- public function getCapabilityCaption($capability) {
+ final public function getCapabilityCaption($capability) {
switch ($capability) {
case PhabricatorPolicyCapability::CAN_VIEW:
if (!$this->canUninstall()) {
@@ -582,7 +582,7 @@
}
}
- public function getCapabilityTemplatePHIDType($capability) {
+ final public function getCapabilityTemplatePHIDType($capability) {
switch ($capability) {
case PhabricatorPolicyCapability::CAN_VIEW:
case PhabricatorPolicyCapability::CAN_EDIT:
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Sun, Oct 20, 12:41 AM (4 w, 5 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6734479
Default Alt Text
D13265.id32114.diff (4 KB)
Attached To
Mode
D13265: Mark some PhabricatorApplication methods as final
Attached
Detach File
Event Timeline
Log In to Comment