diff options
author | mattab <matthieu.aubry@gmail.com> | 2013-07-23 11:52:15 +0400 |
---|---|---|
committer | mattab <matthieu.aubry@gmail.com> | 2013-07-23 11:52:15 +0400 |
commit | 5104d94f3b2250f766b9c520e2da8da9b4cab2e9 (patch) | |
tree | 5f30daf7bc14373fb1bbd0504ce11a771dafc02f /plugins/UsersManager | |
parent | ae4b1f4e38077b174e4df5b7d4513d63fe026a24 (diff) |
Refs #4059 Work in progress: Conversion to use Namespaces of dozen more classes
Removed many Piwik_ functions, in Piwik 2 it is best practise to use the methods calls instead
Todo: finish converting core/ classes + convert plugins/ classes to use \Piwik\Plugin namespace + fix build + Merge master
Diffstat (limited to 'plugins/UsersManager')
-rw-r--r-- | plugins/UsersManager/API.php | 4 | ||||
-rw-r--r-- | plugins/UsersManager/Controller.php | 20 | ||||
-rw-r--r-- | plugins/UsersManager/UsersManager.php | 1 |
3 files changed, 15 insertions, 10 deletions
diff --git a/plugins/UsersManager/API.php b/plugins/UsersManager/API.php index c4c8dc0806..0eeda5be60 100644 --- a/plugins/UsersManager/API.php +++ b/plugins/UsersManager/API.php @@ -485,7 +485,7 @@ class Piwik_UsersManager_API */ public function userExists($userLogin) { - $count = Piwik_FetchOne("SELECT count(*) + $count = Db::fetchOne("SELECT count(*) FROM " . Common::prefixTable("user") . " WHERE login = ?", $userLogin); return $count != 0; @@ -500,7 +500,7 @@ class Piwik_UsersManager_API public function userEmailExists($userEmail) { Piwik::checkUserIsNotAnonymous(); - $count = Piwik_FetchOne("SELECT count(*) + $count = Db::fetchOne("SELECT count(*) FROM " . Common::prefixTable("user") . " WHERE email = ?", $userEmail); return $count != 0 diff --git a/plugins/UsersManager/Controller.php b/plugins/UsersManager/Controller.php index 506c6e6acc..9bc778c6ff 100644 --- a/plugins/UsersManager/Controller.php +++ b/plugins/UsersManager/Controller.php @@ -8,16 +8,20 @@ * @category Piwik_Plugins * @package Piwik_UsersManager */ +use Piwik\API\ResponseBuilder; +use Piwik\Controller\Admin; use Piwik\Piwik; use Piwik\Common; use Piwik\Config; +use Piwik\View; +use Piwik\Url; use Piwik\Site; /** * * @package Piwik_UsersManager */ -class Piwik_UsersManager_Controller extends Piwik_Controller_Admin +class Piwik_UsersManager_Controller extends Admin { static function orderByName($a, $b) { @@ -31,7 +35,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin { Piwik::checkUserIsNotAnonymous(); - $view = new Piwik_View('@UsersManager/index'); + $view = new View('@UsersManager/index'); $IdSitesAdmin = Piwik_SitesManager_API::getInstance()->getSitesIdWithAdminAccess(); $idSiteSelected = 1; @@ -123,7 +127,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin { Piwik::checkUserIsNotAnonymous(); - $view = new Piwik_View('@UsersManager/userSettings'); + $view = new View('@UsersManager/userSettings'); $userLogin = Piwik::getCurrentUserLogin(); if (Piwik::isUserIsSuperUser()) { @@ -164,7 +168,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin $view->ignoreCookieSet = Piwik_Tracker_IgnoreCookie::isIgnoreCookieFound(); $this->initViewAnonymousUserSettings($view); - $view->piwikHost = Piwik_Url::getCurrentHost(); + $view->piwikHost = Url::getCurrentHost(); $this->setBasicVariablesView($view); echo $view->render(); } @@ -181,7 +185,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin /** * The Super User can modify Anonymous user settings - * @param Piwik_View $view + * @param View $view */ protected function initViewAnonymousUserSettings($view) { @@ -226,7 +230,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin */ public function recordAnonymousUserSettings() { - $response = new Piwik_API_ResponseBuilder(Common::getRequestVar('format')); + $response = new ResponseBuilder(Common::getRequestVar('format')); try { Piwik::checkUserIsSuperUser(); $this->checkTokenInUrl(); @@ -253,7 +257,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin */ public function recordUserSettings() { - $response = new Piwik_API_ResponseBuilder(Common::getRequestVar('format')); + $response = new ResponseBuilder(Common::getRequestVar('format')); try { $this->checkTokenInUrl(); @@ -275,7 +279,7 @@ class Piwik_UsersManager_Controller extends Piwik_Controller_Admin } // UI disables password change on invalid host, but check here anyway - if (!Piwik_Url::isValidHost() + if (!Url::isValidHost() && $newPassword !== false ) { throw new Exception("Cannot change password with untrusted hostname!"); diff --git a/plugins/UsersManager/UsersManager.php b/plugins/UsersManager/UsersManager.php index 8d2aeb7a57..3c3b0a761a 100644 --- a/plugins/UsersManager/UsersManager.php +++ b/plugins/UsersManager/UsersManager.php @@ -9,6 +9,7 @@ * @package Piwik_UsersManager */ use Piwik\Piwik; +use Piwik\Piwik_Option; use Piwik\Plugin; /** |