diff options
author | robocoder <anthon.pang@gmail.com> | 2012-03-05 02:04:41 +0400 |
---|---|---|
committer | robocoder <anthon.pang@gmail.com> | 2012-03-05 02:04:41 +0400 |
commit | 884e54cf378bffdea2dd5360af8a260b25352746 (patch) | |
tree | 83fe77732234c1c3e858501ab9ab200989732df1 /plugins/UsersManager | |
parent | 775978937e1e393ba764968d5d41f261b488b8a8 (diff) |
refs #1713 - merge dev branch to trunk (config class refactoring)
git-svn-id: http://dev.piwik.org/svn/trunk@5951 59fd770c-687e-43c8-a1e3-f5a4ff64c105
Diffstat (limited to 'plugins/UsersManager')
-rw-r--r-- | plugins/UsersManager/API.php | 2 | ||||
-rw-r--r-- | plugins/UsersManager/Controller.php | 12 | ||||
-rw-r--r-- | plugins/UsersManager/tests/UsersManager.test.php | 27 |
3 files changed, 26 insertions, 15 deletions
diff --git a/plugins/UsersManager/API.php b/plugins/UsersManager/API.php index c8ac766e19..5b071f468b 100644 --- a/plugins/UsersManager/API.php +++ b/plugins/UsersManager/API.php @@ -622,7 +622,7 @@ class Piwik_UsersManager_API private function checkUserIsNotSuperUser( $userLogin ) { - if($userLogin == Zend_Registry::get('config')->superuser->login) + if($userLogin == Piwik_Config::getInstance()->superuser['login']) { throw new Exception(Piwik_TranslateException("UsersManager_ExceptionSuperUser")); } diff --git a/plugins/UsersManager/Controller.php b/plugins/UsersManager/Controller.php index 63028e8aba..29b29efa96 100644 --- a/plugins/UsersManager/Controller.php +++ b/plugins/UsersManager/Controller.php @@ -108,7 +108,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin $userSettingsDate = Piwik_UsersManager_API::getInstance()->getUserPreference($user, Piwik_UsersManager_API::PREFERENCE_DEFAULT_REPORT_DATE); if($userSettingsDate === false) { - return Zend_Registry::get('config')->General->default_day; + return Piwik_Config::getInstance()->General['default_day']; } return $userSettingsDate; } @@ -127,7 +127,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin { $view->userAlias = $userLogin; $view->userEmail = Piwik::getSuperUserEmail(); - if(!Zend_Registry::get('config')->isFileWritable()) + if(!Piwik_Config::getInstance()->isFileWritable()) { $view->configFileNotWritable = true; } @@ -277,24 +277,24 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin $userLogin = Piwik::getCurrentUserLogin(); if(Piwik::isUserIsSuperUser()) { - $superUser = Zend_Registry::get('config')->superuser; + $superUser = Piwik_Config_Writer::getInstance()->superuser; $updatedSuperUser = false; if($newPassword !== false) { $newPassword = Piwik_Common::unsanitizeInputValue($newPassword); $md5PasswordSuperUser = md5($newPassword); - $superUser->password = $md5PasswordSuperUser; + $superUser['password'] = $md5PasswordSuperUser; $updatedSuperUser = true; } if($superUser->email != $email) { - $superUser->email = $email; + $superUser['email'] = $email; $updatedSuperUser = true; } if($updatedSuperUser) { - Zend_Registry::get('config')->superuser = $superUser->toArray(); + Piwik_Config::getInstance()->superuser = $superUser; } } else diff --git a/plugins/UsersManager/tests/UsersManager.test.php b/plugins/UsersManager/tests/UsersManager.test.php index 8641c08a79..af3a6e6629 100644 --- a/plugins/UsersManager/tests/UsersManager.test.php +++ b/plugins/UsersManager/tests/UsersManager.test.php @@ -8,26 +8,37 @@ require_once PIWIK_PATH_TEST_TO_ROOT . '/tests/core/Database.test.php'; class Test_Piwik_UsersManager extends Test_Database { - function setUp() - { - parent::setUp(); + function __construct() + { + parent::__construct(); + + Piwik_Config::getInstance()->setTestEnvironment(); + Piwik_PluginsManager::getInstance()->unloadPlugins(); + Piwik_PluginsManager::getInstance()->loadPlugins( array('LanguagesManager') ); + $plugin = Piwik_PluginsManager::getInstance()->getLoadedPlugin('LanguagesManager'); + $plugin->install(); + } + + function setUp() + { + parent::setUp(); // setup the access layer - $pseudoMockAccess = new FakeAccess; + $pseudoMockAccess = new FakeAccess; FakeAccess::setIdSitesView( array(1,2)); FakeAccess::setIdSitesAdmin( array(3,4)); //finally we set the user as a super user by default FakeAccess::$superUser = true; Zend_Registry::set('access', $pseudoMockAccess); - + // we make sure the tests don't depend on the config file content - Zend_Registry::get('config')->superuser = array( + Piwik_Config::getInstance()->superuser = array( 'login'=>'superusertest', 'password'=>'passwordsuperusertest', 'email'=>'superuser@example.com' ); - } + } private function _flatten($sitesAccess) { @@ -65,7 +76,7 @@ class Test_Piwik_UsersManager extends Test_Database function test_all_superUserIncluded() { - Zend_Registry::get('config')->superuser = array( + Piwik_Config::getInstance()->superuser = array( 'login'=>'superusertest', 'password'=>'passwordsuperusertest', 'email'=>'superuser@example.com' |