diff --git a/core/Access.php b/core/Access.php index e404e69d947..7d35ed72cf5 100644 --- a/core/Access.php +++ b/core/Access.php @@ -249,7 +249,6 @@ public function hasSuperUserAccess() /** * @see Access::hasSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ public function isSuperUser() { @@ -345,7 +344,6 @@ public function checkUserHasSuperUserAccess() /** * @see Access::checkUserHasSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ public function checksUserIsSuperUser() { diff --git a/core/Piwik.php b/core/Piwik.php index b3c91a6aa2d..000ad8dd8f9 100644 --- a/core/Piwik.php +++ b/core/Piwik.php @@ -288,7 +288,6 @@ static public function hasUserSuperUserAccessOrIsTheUser($theUser) /** * @see Piwik::hasUserSuperUserAccessOrIsTheUser() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ static public function isUserIsSuperUserOrTheUser($theUser) { @@ -298,7 +297,6 @@ static public function isUserIsSuperUserOrTheUser($theUser) /** * @see Piwik::checkUserHasSuperUserAccessOrIsTheUser() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ static public function checkUserIsSuperUserOrTheUser($theUser) { @@ -354,7 +352,6 @@ static public function hasTheUserSuperUserAccess($theUser) /** * @see Piwik::hasUserSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ static public function isUserIsSuperUser() { @@ -420,7 +417,6 @@ static public function setUserHasSuperUserAccess($bool = true) /** * @see Piwik::setUserHasSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ static public function setUserIsSuperUser($bool = true) { @@ -430,7 +426,6 @@ static public function setUserIsSuperUser($bool = true) /** * @see Piwik::checkUserHasSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ static public function checkUserIsSuperUser() { diff --git a/plugins/UsersManager/API.php b/plugins/UsersManager/API.php index 0336e26fe76..87593b2ed65 100644 --- a/plugins/UsersManager/API.php +++ b/plugins/UsersManager/API.php @@ -582,7 +582,7 @@ public function setUserAccess($userLogin, $access, $idSites) { $this->checkAccessType($access); $this->checkUserExists($userLogin); - $this->checkUserIsNotConfigSuperUser($userLogin); + $this->checkUserHasNotSuperUserAccess($userLogin); if ($userLogin == 'anonymous' && $access == 'admin' @@ -660,6 +660,13 @@ private function checkUserIsNotAnonymous($userLogin) } } + private function checkUserHasNotSuperUserAccess($userLogin) + { + if (Piwik::hasTheUserSuperUserAccess($userLogin)) { + throw new Exception(Piwik::translate("UsersManager_ExceptionSuperUser")); + } + } + private function checkUserIsNotConfigSuperUser($userLogin) { if ($userLogin == Piwik::getConfigSuperUserLogin()) { diff --git a/plugins/UsersManager/Controller.php b/plugins/UsersManager/Controller.php index 6b26c168fad..3ab739e7005 100644 --- a/plugins/UsersManager/Controller.php +++ b/plugins/UsersManager/Controller.php @@ -88,7 +88,7 @@ function index() } $superUsers = array(); - if (Piwik::isUserHasSomeAdminAccess()) { + if (Piwik::hasUserSuperUserAccess()) { $users = APIUsersManager::getInstance()->getUsers(); foreach ($users as $user) { if ($user['superuser_access']) { diff --git a/tests/PHPUnit/FakeAccess.php b/tests/PHPUnit/FakeAccess.php index 19e00ab38f2..45c703b545c 100644 --- a/tests/PHPUnit/FakeAccess.php +++ b/tests/PHPUnit/FakeAccess.php @@ -54,7 +54,6 @@ public static function checkUserHasSuperUserAccess() /** * @see FakeAccess::checkUserHasSuperUserAccess() * @deprecated deprecated since version 2.0.4 - * @todo To be removed from April 1st 2014. */ public function checksUserIsSuperUser() {