Applied PR feedback

AltamashShaikh · AltamashShaikh · commit bfb4da07e45d · 2025-05-28T07:20:39.000+05:30
diff --git a/Google/Authorization.php b/Google/Authorization.php
@@ -137,7 +137,7 @@ protected function getUserInfoByAccessToken(\Matomo\Dependencies\GoogleAnalytics
     private function getValidUri($uris)
     {
         $validUri = Url::getCurrentUrlWithoutQueryString() . '?module=GoogleAnalyticsImporter&action=processAuthCode';
-        $validUriFallback = SettingsPiwik::getPiwikUrl() . '?module=GoogleAnalyticsImporter&action=processAuthCode'; // Some MWP installs was not working as expected when using Url::getCurrentUrlWithoutQueryString()
+        $validUriFallback = SettingsPiwik::getPiwikUrl() . 'index.php?module=GoogleAnalyticsImporter&action=processAuthCode'; // Some MWP installs was not working as expected when using Url::getCurrentUrlWithoutQueryString()
         foreach ($uris as $uri) {
             if (stripos($uri, $validUri) !== \false || stripos($uri, $validUriFallback) !== \false) {
                 return $uri;

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ protected function getUserInfoByAccessToken(\Matomo\Dependencies\GoogleAnalytics`
`137`	`137`	`private function getValidUri($uris)`
`138`	`138`	`{`
`139`	`139`	`$validUri = Url::getCurrentUrlWithoutQueryString() . '?module=GoogleAnalyticsImporter&action=processAuthCode';`
`140`		`- $validUriFallback = SettingsPiwik::getPiwikUrl() . '?module=GoogleAnalyticsImporter&action=processAuthCode'; // Some MWP installs was not working as expected when using Url::getCurrentUrlWithoutQueryString()`
	`140`	`+ $validUriFallback = SettingsPiwik::getPiwikUrl() . 'index.php?module=GoogleAnalyticsImporter&action=processAuthCode'; // Some MWP installs was not working as expected when using Url::getCurrentUrlWithoutQueryString()`
`141`	`141`	`foreach ($uris as $uri) {`
`142`	`142`	`if (stripos($uri, $validUri) !== \false \|\| stripos($uri, $validUriFallback) !== \false) {`
`143`	`143`	`return $uri;`