Full refactor of site content detection and no data page logic

plugins/CoreAdminHome/Tasks.php

@@ -147,7 +147,7 @@ public function checkSiteHasTrackedVisits($idSite)
     {
         $this->rememberTrackingCodeReminderRan($idSite);
 
-        if (!SitesManager::shouldPerormEmptySiteCheck($idSite)) {
+        if (!SitesManager::shouldPerformEmptySiteCheck($idSite)) {
             return;
         }
 

plugins/CoreAdminHome/stylesheets/trackingCodeGenerator.less

@@ -18,7 +18,7 @@
   }
 }
 
-#tracking-code li .trackingCodeAdvancedOptions .advance-option {
+.site-without-data #matomo li .trackingCodeAdvancedOptions .advance-option {
   margin-top: 1rem;
   margin-left: 1.25rem
 }

plugins/CoreAdminHome/templates/trackingCodeGenerator.twig

@@ -25,7 +25,7 @@
             <a href="#/trackingApi">{{ 'CoreAdminHome_HttpTrackingApi'|translate }}</a>
             <a href="#/singlePageApplication">{{ 'SitesManager_SiteWithoutDataSinglePageApplication'|translate }}</a>
             <a href="#/google-tag-manager">{{ 'SitesManager_SiteWithoutDataGoogleTagManager'|translate }}</a>
-            <a href="#/wordpress">Wordpress</a>
+            <a href="#/wordpress">WordPress</a>
             <a href="#/cloudflare">Cloudflare</a>
             <a href="#/vue">Vue.js</a>
             <a href="#/react">React.js</a>
@@ -71,7 +71,7 @@
     <p>{{ 'CoreAdminHome_GoogleTagManagerDescription'|translate('<a href="https://matomo.org/faq/new-to-piwik/how-do-i-use-matomo-analytics-within-gtm-google-tag-manager/" rel="noreferrer noopener" target="_blank">','</a>')|raw }}</p>
 </div>
 
-<div vue-entry="CoreHome.ContentBlock" content-title="Wordpress" anchor="wordpress">
+<div vue-entry="CoreHome.ContentBlock" content-title="WordPress" anchor="wordpress">
     <p>{{ 'CoreAdminHome_WordpressDescription'|translate('<a href="https://matomo.org/faq/new-to-piwik/how-do-i-install-the-matomo-tracking-code-on-wordpress/" target="_blank">','</a>')|raw }}</p>
 </div>
 

plugins/CoreAdminHome/vue/src/JsTrackingCodeGenerator/JsTrackingCodeGenerator.vue

@@ -158,7 +158,7 @@ export default defineComponent({
         {
           module: 'API',
           format: 'json',
-          method: 'Tour.detectConsentManager',
+          method: 'SitesManager.detectConsentManager',
           idSite,
           filter_limit: '-1',
         },

plugins/Installation/Controller.php

@@ -26,6 +26,7 @@
 use Piwik\Plugins\Diagnostics\DiagnosticService;
 use Piwik\Plugins\LanguagesManager\LanguagesManager;
 use Piwik\Plugins\SitesManager\API as APISitesManager;
+use Piwik\Plugins\SitesManager\SitesManager;
 use Piwik\Plugins\UsersManager\API as APIUsersManager;
 use Piwik\Plugins\UsersManager\NewsletterSignup;
 use Piwik\Plugins\UsersManager\UserUpdater;
@@ -393,29 +394,13 @@ public function trackingCode()
 
         $javascriptGenerator = new TrackerCodeGenerator();
         $jsTag = $javascriptGenerator->generate($idSite, Url::getCurrentUrlWithoutFileName());
-        $rawJsTag = TrackerCodeGenerator::stripTags($jsTag);
-
-        $showMatomoLinks = true;
-        Piwik::postEvent('SitesManager.showMatomoLinksInTrackingCodeEmail', array(&$showMatomoLinks));
-
-        $trackingUrl = trim(SettingsPiwik::getPiwikUrl(), '/') . '/' . $javascriptGenerator->getPhpTrackerEndpoint();
-
-        $this->siteContentDetector->detectContent([SiteContentDetector::ALL_CONTENT]);
-
-        $emailBody = $this->renderTemplateAs('@SitesManager/_trackingCodeEmail', array(
-            'jsTag' => $rawJsTag,
-            'showMatomoLinks' => $showMatomoLinks,
-            'trackingUrl' => $trackingUrl,
-            'idSite' => $idSite,
-            'gtmUsed' => $this->siteContentDetector->gtm,
-            'ga3Used' => $this->siteContentDetector->ga3,
-            'ga4Used' => $this->siteContentDetector->ga4,
-            'cloudflare' => $this->siteContentDetector->cloudflare,
-            'jsFramework' => $this->siteContentDetector->jsFramework,
-            'consentManagerName' => $this->siteContentDetector->consentManagerName,
-            'consentManagerUrl' => $this->siteContentDetector->consentManagerUrl,
-            'consentManagerIsConnected' => $this->siteContentDetector->isConnected
-        ), $viewType = 'basic');
+
+        // Needs to be generated as super user, as API requests would otherwise fail
+        $emailBody = Access::doAsSuperUser(
+            function () use ($idSite) {
+                return SitesManager::renderTrackingCodeEmail($idSite);
+            }
+        );
 
         // Load the Tracking code and help text from the SitesManager
         $viewTrackingHelp = new \Piwik\View('@SitesManager/_displayJavascriptCode');
@@ -426,15 +411,6 @@ public function trackingCode()
         $viewTrackingHelp->piwikUrl = Url::getCurrentUrlWithoutFileName();
         $viewTrackingHelp->isInstall = true;
 
-        $viewTrackingHelp->gtmUsed = $this->siteContentDetector->gtm;
-        $viewTrackingHelp->ga3Used = $this->siteContentDetector->ga3;
-        $viewTrackingHelp->ga4Used = $this->siteContentDetector->ga4;
-        $viewTrackingHelp->cloudflare = $this->siteContentDetector->cloudflare;
-        $viewTrackingHelp->jsFramework = $this->siteContentDetector->jsFramework;
-        $viewTrackingHelp->consentManagerName = $this->siteContentDetector->consentManagerName;
-        $viewTrackingHelp->consentManagerUrl = $this->siteContentDetector->consentManagerUrl;
-        $viewTrackingHelp->consentManagerIsConnected = $this->siteContentDetector->isConnected;
-
         $view->trackingHelp = $viewTrackingHelp->render();
         $view->displaySiteName = $siteName;
 

plugins/PrivacyManager/Controller.php

@@ -21,6 +21,8 @@
 use Piwik\Plugins\CustomJsTracker\File;
 use Piwik\Plugins\LanguagesManager\LanguagesManager;
 use Piwik\Plugins\LanguagesManager\API as APILanguagesManager;
+use Piwik\Plugins\SitesManager\SiteContentDetection\ConsentManagerDetectionAbstract;
+use Piwik\Plugins\SitesManager\SiteContentDetection\SiteContentDetectionAbstract;
 use Piwik\SiteContentDetector;
 use Piwik\Scheduler\Scheduler;
 use Piwik\Tracker\TrackerCodeGenerator;
@@ -175,16 +177,23 @@ public function consent()
 
         $view = new View('@PrivacyManager/askingForConsent');
 
-        $this->siteContentDetector->detectContent([SiteContentDetector::CONSENT_MANAGER]);
+        $this->siteContentDetector->detectContent([SiteContentDetectionAbstract::TYPE_CONSENT_MANAGER]);
+        $consentManager = $this->siteContentDetector->getDetectsByType(SiteContentDetectionAbstract::TYPE_CONSENT_MANAGER);
         $view->consentManagerName = null;
-        if ($this->siteContentDetector->consentManagerId) {
-            $view->consentManagerName = $this->siteContentDetector->consentManagerName;
-            $view->consentManagerUrl = $this->siteContentDetector->consentManagerUrl;
-            $view->consentManagerIsConnected = $this->siteContentDetector->isConnected;
+        if (!empty($consentManager)) {
+            $consentManager = $this->siteContentDetector->getSiteContentDetectionById(reset($consentManager));
+            if ($consentManager instanceof ConsentManagerDetectionAbstract) {
+                $view->consentManagerName = $consentManager::getName();
+                $view->consentManagerUrl = $consentManager::getInstructionUrl();
+                $view->consentManagerIsConnected = in_array(
+                    $consentManager::getId(),
+                    $this->siteContentDetector->connectedConsentManagers
+                );
+            }
         }
 
-        $consentManagers = SiteContentDetector::getConsentManagerDefinitions();
-        $knownConsentManagers = array_combine(array_column($consentManagers, 'name'), array_column($consentManagers, 'url'));
+        $consentManagers = SiteContentDetector::getKnownConsentManagers();
+        $knownConsentManagers = array_combine(array_column($consentManagers, 'name'), array_column($consentManagers, 'instructionUrl'));
 
         $view->knownConsentManagers = $knownConsentManagers;
         $this->setBasicVariablesView($view);

plugins/SitesManager/API.php

@@ -24,6 +24,8 @@
 use Piwik\Piwik;
 use Piwik\Plugin\SettingsProvider;
 use Piwik\Plugins\CorePluginsAdmin\SettingsMetadata;
+use Piwik\Plugins\SitesManager\SiteContentDetection\ConsentManagerDetectionAbstract;
+use Piwik\Plugins\SitesManager\SiteContentDetection\SiteContentDetectionAbstract;
 use Piwik\Plugins\WebsiteMeasurable\Settings\Urls;
 use Piwik\ProxyHttp;
 use Piwik\Scheduler\Scheduler;
@@ -32,6 +34,7 @@
 use Piwik\SettingsPiwik;
 use Piwik\SettingsServer;
 use Piwik\Site;
+use Piwik\SiteContentDetector;
 use Piwik\Tracker\Cache;
 use Piwik\Tracker\TrackerCodeGenerator;
 use Piwik\Translation\Translator;
@@ -84,11 +87,15 @@ class API extends \Piwik\Plugin\API
 
     private $timezoneNameCache = [];
 
-    public function __construct(SettingsProvider $provider, SettingsMetadata $settingsMetadata, Translator $translator)
+    /** @var SiteContentDetector */
+    private $siteContentDetector;
+
+    public function __construct(SettingsProvider $provider, SettingsMetadata $settingsMetadata, Translator $translator, SiteContentDetector $siteContentDetector)
     {
         $this->settingsProvider = $provider;
         $this->settingsMetadata = $settingsMetadata;
         $this->translator = $translator;
+        $this->siteContentDetector = $siteContentDetector;
     }
 
     /**
@@ -1764,4 +1771,29 @@ public function getNumWebsitesToDisplayPerPage()
 
         return SettingsPiwik::getWebsitesCountToDisplay();
     }
+
+
+    /**
+     * Detect consent manager details for a site
+     *
+     * @internal
+     * @unsanitized
+     */
+    public function detectConsentManager(int $idSite, int $timeOut = 60): ?array
+    {
+        Piwik::checkUserHasViewAccess($idSite);
+
+        $this->siteContentDetector->detectContent([SiteContentDetectionAbstract::TYPE_CONSENT_MANAGER], $idSite, null, $timeOut);
+        $consentManagers = $this->siteContentDetector->getDetectsByType(SiteContentDetectionAbstract::TYPE_CONSENT_MANAGER);
+        if (!empty($consentManagers)) {
+            /** @var ConsentManagerDetectionAbstract $consentManager */
+            $consentManager = $this->siteContentDetector->getSiteContentDetectionById(reset($consentManagers));
+            return ['name' => $consentManager::getName(),
+                    'url' => $consentManager::getInstructionUrl(),
+                    'isConnected' => in_array($consentManager::getId(), $this->siteContentDetector->connectedConsentManagers)
+            ];
+        }
+
+        return null;
+    }
 }