Fix possible error when generating reports for custom dimensions

core/Archive.php

@@ -454,7 +454,7 @@ public static function createDataTableFromArchive($recordName, $idSite, $period,
         }
 
         $archive = Archive::build($idSite, $period, $date, $segment, $_restrictSitesToLogin = false);
-        if ($idSubtable === false) {
+        if ($idSubtable === false || $idSubtable === '') {
             $idSubtable = null;
         }
 

plugins/CustomDimensions/API.php

@@ -47,11 +47,11 @@ class API extends \Piwik\Plugin\API
      * @param bool|false $segment
      * @param bool|false $expanded
      * @param bool|false $flat
-     * @param int|null   $idSubtable
+     * @param int|false $idSubtable
      * @return DataTable|DataTable\Map
      * @throws \Exception
      */
-    public function getCustomDimension($idDimension, $idSite, $period, $date, $segment = false, $expanded = false, $flat = false, $idSubtable = null)
+    public function getCustomDimension($idDimension, $idSite, $period, $date, $segment = false, $expanded = false, $flat = false, $idSubtable = false)
     {
         Piwik::checkUserHasViewAccess($idSite);
 
@@ -62,7 +62,7 @@ public function getCustomDimension($idDimension, $idSite, $period, $date, $segme
 
         $dataTable = Archive::createDataTableFromArchive($record, $idSite, $period, $date, $segment, $expanded, $flat, $idSubtable);
 
-        if (isset($idSubtable) && $dataTable->getRowsCount()) {
+        if (!empty($idSubtable) && $dataTable->getRowsCount()) {
             $parentTable = Archive::createDataTableFromArchive($record, $idSite, $period, $date, $segment);
             foreach ($parentTable->getRows() as $row) {
                 if ($row->getIdSubDataTable() == $idSubtable) {

plugins/CustomDimensions/tests/Fixtures/TrackVisitsWithCustomDimensionsFixture.php

@@ -7,13 +7,16 @@
  */
 namespace Piwik\Plugins\CustomDimensions\tests\Fixtures;
 
+use Piwik\Context;
 use Piwik\Date;
 use Piwik\Plugins\CustomDimensions\CustomDimensions;
 use Piwik\Plugins\CustomDimensions\Dao\Configuration;
 use Piwik\Plugins\CustomDimensions\Dimension\Extraction;
 use Piwik\Plugins\Goals;
+use Piwik\Plugins\ScheduledReports\API as APIScheduledReports;
+use Piwik\Plugins\ScheduledReports\ScheduledReports;
+use Piwik\ReportRenderer;
 use Piwik\Tests\Framework\Fixture;
-use Piwik\Plugin;
 use Piwik\Tracker\Cache;
 
 /**
@@ -32,6 +35,7 @@ public function setUp(): void
         $this->setUpWebsites();
         $this->addGoals();
         $this->configureSomeDimensions();
+        $this->configureScheduledReport();
         $this->trackFirstVisit();
         $this->trackSecondVisit();
         $this->trackThirdVisit();
@@ -77,6 +81,23 @@ private function configureSomeDimensions()
         Cache::clearCacheGeneral();
     }
 
+    protected function configureScheduledReport()
+    {
+        // Context change is needed, as adding the custom dimensions reports looks for the idSite in the request params
+        Context::changeIdSite(1, function() {
+            APIScheduledReports::getInstance()->addReport(
+                $idSite = 1,
+                'ScheduledReport',
+                'month',
+                0,
+                ScheduledReports::EMAIL_TYPE,
+                ReportRenderer::PDF_FORMAT,
+                ['VisitsSummary_get', 'CustomDimensions_getCustomDimension_idDimension--1', 'CustomDimensions_getCustomDimension_idDimension--2'],
+                [ScheduledReports::DISPLAY_FORMAT_PARAMETER => ScheduledReports::DISPLAY_FORMAT_TABLES_AND_GRAPHS]
+            );
+        });
+    }
+
     protected function trackFirstVisit()
     {
         $t = self::getTracker($this->idSite, $this->dateTime, $defaultInit = true);

plugins/CustomDimensions/tests/System/ApiTest.php

@@ -8,7 +8,9 @@
 
 namespace Piwik\Plugins\CustomDimensions\tests\System;
 
+use Piwik\Context;
 use Piwik\Plugins\CustomDimensions\tests\Fixtures\TrackVisitsWithCustomDimensionsFixture;
+use Piwik\ReportRenderer;
 use Piwik\Tests\Framework\TestCase\SystemTestCase;
 
 /**
@@ -260,6 +262,26 @@ public function getApiForTesting()
         return $apiToTest;
     }
 
+    public function testScheduledReport()
+    {
+        // Context change is needed, as otherwise the customdimension reports are not available
+        Context::changeIdSite(1, function(){
+            $this->runApiTests(['ScheduledReports.generateReport'], [
+                'idSite'                 => 1,
+                'date'                   => self::$fixture->dateTime,
+                'periods'                => ['year'],
+                'format'                 => 'original',
+                'fileExtension'          => 'pdf',
+                'otherRequestParameters' => [
+                    'idReport'     => 1,
+                    'reportFormat' => ReportRenderer::PDF_FORMAT,
+                    'outputType'   => \Piwik\Plugins\ScheduledReports\API::OUTPUT_RETURN,
+                    'serialize'    => 0,
+                ],
+            ]);
+        });
+    }
+
     public static function getOutputPrefix()
     {
         return '';