revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-19 05:12:41 +01:00
Code Issues Releases Wiki Activity

Remove some no-op "canUninstall()" Application methods

Browse source 
Summary: The default behavior of these methods is to return `true`, so these overrides have no effect.

Test Plan: `grep`; poked around.

Reviewers: chad

Reviewed By: chad

Subscribers: hach-que

Differential Revision: https://secure.phabricator.com/D16985
This commit is contained in:
epriestley 2016-12-05 09:46:39 -08:00
parent 5f593aafb1
commit 4a6229ee69
3 changed files with 0 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/badges/application/PhabricatorBadgesApplication.php
View file

@ -26,10 +26,6 @@ final class PhabricatorBadgesApplication extends PhabricatorApplication {
return self::GROUP_UTILITIES; return self::GROUP_UTILITIES;
} }
public function canUninstall() {
return true;
}
public function isPrototype() { public function isPrototype() {
return true; return true;
} }

4
src/applications/nuance/application/PhabricatorNuanceApplication.php
View file

@ -23,10 +23,6 @@ final class PhabricatorNuanceApplication extends PhabricatorApplication {
return false; return false;
} }
public function canUninstall() {
return true;
}
public function getBaseURI() { public function getBaseURI() {
return '/nuance/'; return '/nuance/';
} }

4
src/applications/phragment/application/PhabricatorPhragmentApplication.php
View file

@ -30,10 +30,6 @@ final class PhabricatorPhragmentApplication extends PhabricatorApplication {
return true; return true;
} }
public function canUninstall() {
return true;
}
public function getRoutes() { public function getRoutes() {
return array( return array(
'/phragment/' => array( '/phragment/' => array(