diff --git a/plugins/SitesManager/API.php b/plugins/SitesManager/API.php index 933ba5f18640598a581b9ef35c38cc9d52b1df7c..e6505320ad6c6cf5cb3930f97a19356994f40d86 100644 --- a/plugins/SitesManager/API.php +++ b/plugins/SitesManager/API.php @@ -418,6 +418,7 @@ class API extends \Piwik\Plugin\API // return all variations of the URL return array( + $url, "http://" . $hostname, "http://www." . $hostname, "https://" . $hostname, diff --git a/plugins/SitesManager/tests/Integration/ApiTest.php b/plugins/SitesManager/tests/Integration/ApiTest.php index 6f85c3abd75a958010cbb9daece4ab59f8ef4577..011ff7a344b137994f264dcadbfbbab395834321 100644 --- a/plugins/SitesManager/tests/Integration/ApiTest.php +++ b/plugins/SitesManager/tests/Integration/ApiTest.php @@ -1004,22 +1004,21 @@ class ApiTest extends IntegrationTestCase public function test_getSitesIdFromSiteUrl_matchesBothHttpAndHttpsUrls_asSuperUser() { - API::getInstance()->addSite("site1", array("https://piwik.org", "http://example.com")); + API::getInstance()->addSite("site1", array("https://piwik.org", "http://example.com", "fb://special-url")); $this->assert_getSitesIdFromSiteUrl_matchesBothHttpAndHttpsUrls(); } public function test_getSitesIdFromSiteUrl_matchesBothHttpAndHttpsUrls_asUserWithViewPermission() { - API::getInstance()->addSite("site1", array("https://piwik.org", "http://example.com")); + API::getInstance()->addSite("site1", array("https://piwik.org", "http://example.com", "fb://special-url")); APIUsersManager::getInstance()->addUser("user1", "geqgegagae", "tegst@tesgt.com", "alias"); APIUsersManager::getInstance()->setUserAccess("user1", "view", array(1)); + // Make sure we're not Super user FakeAccess::setSuperUserAccess(false); FakeAccess::$identity = 'user1'; - - // Make sure we're not Super user $this->assertFalse(Piwik::hasUserSuperUserAccess()); $this->assert_getSitesIdFromSiteUrl_matchesBothHttpAndHttpsUrls(); @@ -1039,6 +1038,9 @@ class ApiTest extends IntegrationTestCase $idsites = API::getInstance()->getSitesIdFromSiteUrl('https://example.com'); $this->assertTrue(count($idsites) == 1); + $idsites = API::getInstance()->getSitesIdFromSiteUrl("fb://special-url"); + $this->assertTrue(count($idsites) == 1); + $idsites = API::getInstance()->getSitesIdFromSiteUrl('https://random-example.com'); $this->assertTrue(count($idsites) == 0);