Doesn't this multiply the number of archive queries by number of sites?

That might be possible. Not sure if all archives are fetched with one query if idSites=all is given to an API method...
But it at least includes all reports. Fetching the report metadata with idSites=all excludes various reports as they don't support it.

That might be possible. Not sure if all archives are fetched with one query if idSites=all is given to an API method...

If multiple idSites are given to Archive::get(, it should be just one query using idsite IN (. Otherwise, it's a couple queries per idSite.

Ok. Alternatively we would need to make all API methods and reports metadata compatible with multiple idSites. Tried catching errors in the first place instead of iterating through the sites, but that removes a lot of metrics, as the reports aren't available for multiple sites...

@sgiehl @diosmosis we won't be fixing this for now I reckon as it's not a priority and we'll need to see re performance impact etc. Also the API call to getREportMetadata would need to be executed through Request::processRequest potentially etc re segments/dimensions and other things that might be site specific. If we continue to support this feature then we'd likely would need to fix this maybe in the plugins itself or so.

fyi documented this for now in matomo-org/developer-documentation#377

I've created an issue for now in #16366 and we'll schedule this work eventually to see if other APIs are impacted too and what a proper fix be should we keep this feature. I'll close the PR for now but we might reopen it again later.