[Bug] Matomo Query Report Custom Range doesn't work

[bozzelliandrea]

What happened?

Using the Archive class to retrieve archived blob storage data doesn't work in a custom date range period.
If I filter the data using day/week/month/year as period, the data aggregation work as expected.

With data range period, starting date monday of the week, ending date sunday, returned dataTable is empty.

Same query with week as period, dataTable returns all expected counters.

What should happen?

Expected blob data aggregated and returned from Archive class.

How can this be reproduced?

• Create and prepare an Archiver and generate archive blob records
• Get archived data with range date as period

 $period = \Piwik\Common::getRequestVar('period', 'day', 'string');
 $date = \Piwik\Common::getRequestVar('date', 'today', 'string');
 // $period = range
 $archive = Archive::build($idSite, $period, $date);

 $archiveDataTable = $archive->getDataTable('Custom_Report'); // expected counters here

Matomo version

2.5.0 on premise

PHP version

8.1

Server operating system

No response

What browsers are you seeing the problem on?

No response

Computer operating system

No response

Relevant log output

No response

Validations

• Read our Contributing Guidelines.
• Follow our Security Policy.
• Check that there isn't already an issue that reports the same bug to avoid creating duplicates.
• The provided steps to reproduce is a minimal reproducible of the Bug.

[bozzelliandrea]

just a note about the issue:
following the docs at https://developer.matomo.org/guides/archiving the problem seems the if condition using the function, when a request is executed with date range, the if statement is not executed. By removing the if statement it is correctly executed also in the ranges and the data displayed

if ($this->isRequestedReport('MyPlugin_mySpecificReport')) {
      $this->getProcessor()->aggregateDataTableRecords(['MyPlugin_mySpecificReport']);
  }

[sgiehl]

@bozzelliandrea The way archiving should be implemented is actually about to change. There is an open PR for the documentation to add that: matomo-org/developer-documentation#739
We though still need to update all existing stuff in matomo-org/developer-documentation#733