diff options
author | Stefan Giehl <stefan@piwik.org> | 2018-10-03 22:52:29 +0300 |
---|---|---|
committer | diosmosis <diosmosis@users.noreply.github.com> | 2018-10-03 22:52:29 +0300 |
commit | 902bd940c87fd9f0d1730dc2b9560b862b8c8e9a (patch) | |
tree | ea03fcb6f0a79f8e6f06b2814651d55b91f306ea /plugins/Insights | |
parent | 67eaa2c0381933d5e38180a2159044893447acca (diff) |
Improve usage of requested site in plugin Controllers (#13526)
Diffstat (limited to 'plugins/Insights')
-rw-r--r-- | plugins/Insights/Controller.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugins/Insights/Controller.php b/plugins/Insights/Controller.php index 97564dcbef..ace08c612a 100644 --- a/plugins/Insights/Controller.php +++ b/plugins/Insights/Controller.php @@ -22,9 +22,10 @@ class Controller extends \Piwik\Plugin\Controller public function __construct() { - $idSite = Common::getRequestVar('idSite', null, 'int'); + parent::__construct(); - Piwik::checkUserHasViewAccess($idSite); + $this->checkSitePermission(); + Piwik::checkUserHasViewAccess($this->idSite); } public function getInsightsOverview() @@ -68,12 +69,11 @@ class Controller extends \Piwik\Plugin\Controller return; } - $idSite = Common::getRequestVar('idSite', null, 'int'); $period = Common::getRequestVar('period', null, 'string'); $date = Common::getRequestVar('date', null, 'string'); $segment = Request::getRawSegmentFromRequest(); - return API::getInstance()->$apiReport($idSite, $period, $date, $segment); + return API::getInstance()->$apiReport($this->idSite, $period, $date, $segment); } private function canGenerateInsights() |