Skip to content
Extraits de code Groupes Projets
Valider aa39881b rédigé par mattab's avatar mattab
Parcourir les fichiers

Small refactor

parent 2ba435ab
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
...@@ -592,7 +592,7 @@ abstract class Controller ...@@ -592,7 +592,7 @@ abstract class Controller
$view->isValidHost = Url::isValidHost(); $view->isValidHost = Url::isValidHost();
if (!$view->isValidHost) { if (!$view->isValidHost) {
// invalid host, so display warning to user // invalid host, so display warning to user
$validHosts = Url::getTrustedHosts($filterEnrich = false); $validHosts = Url::getTrustedHostsFromConfig();
$validHost = $validHosts[0]; $validHost = $validHosts[0];
$invalidHost = Common::sanitizeInputValue($_SERVER['HTTP_HOST']); $invalidHost = Common::sanitizeInputValue($_SERVER['HTTP_HOST']);
......
...@@ -537,11 +537,10 @@ class Url ...@@ -537,11 +537,10 @@ class Url
&& in_array($parsedUrl['scheme'], array('http', 'https')); && in_array($parsedUrl['scheme'], array('http', 'https'));
} }
public static function getTrustedHosts( $filterEnrich = true ) public static function getTrustedHostsFromConfig()
{ {
$trustedHosts = @Config::getInstance()->General['trusted_hosts']; $trustedHosts = @Config::getInstance()->General['trusted_hosts'];
if (!is_array($trustedHosts)) {
if (empty($trustedHosts)) {
return array(); return array();
} }
foreach ($trustedHosts as &$trustedHost) { foreach ($trustedHosts as &$trustedHost) {
...@@ -550,11 +549,15 @@ class Url ...@@ -550,11 +549,15 @@ class Url
$trustedHost = parse_url($trustedHost, PHP_URL_HOST); $trustedHost = parse_url($trustedHost, PHP_URL_HOST);
} }
} }
return $trustedHosts;
}
if($filterEnrich) { public static function getTrustedHosts()
/* used by Piwik PRO */ {
Piwik::postEvent('Url.filterTrustedHosts', array(&$trustedHosts)); $trustedHosts = self::getTrustedHostsFromConfig();
}
/* used by Piwik PRO */
Piwik::postEvent('Url.filterTrustedHosts', array(&$trustedHosts));
return $trustedHosts; return $trustedHosts;
} }
......
...@@ -50,7 +50,7 @@ class Controller extends \Piwik\Plugin\ControllerAdmin ...@@ -50,7 +50,7 @@ class Controller extends \Piwik\Plugin\ControllerAdmin
if (Piwik::hasUserSuperUserAccess()) { if (Piwik::hasUserSuperUserAccess()) {
$this->handleGeneralSettingsAdmin($view); $this->handleGeneralSettingsAdmin($view);
$view->trustedHosts = Url::getTrustedHosts( $filterEnrich = false ); $view->trustedHosts = Url::getTrustedHostsFromConfig();
$logo = new CustomLogo(); $logo = new CustomLogo();
$view->branding = array('use_custom_logo' => $logo->isEnabled()); $view->branding = array('use_custom_logo' => $logo->isEnabled());
......
0% Chargement en cours ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter