- Issue created by @Rajab Natshah
- @rajab-natshah opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 7:54pm 14 March 2023 - 🇩🇪Germany Erik Seifert
I think matomo query do not have a accessCheck. https://git.drupalcode.org/project/dashboards/-/blob/39b94da4ea4f048770b...
- 🇯🇴Jordan Rajab Natshah Jordan
Thanks, Erik for the hint.
Faced the issue of ✨ Add Drupal 10 compatibility to Matomo reporting API Needs review
I was not able to physically test that in Drupal 10.Do you mean it should be moved to before the
execute()
+ $query->accessCheck(FALSE); $query->setParameters($params); $response = $query->execute()->getRawResponse();
Spotted it now.
use Matomo\ReportingApi\QueryFactory;
https://git.drupalcode.org/project/matomo_reporting_api/-/blob/8.x-2.x/s...
It is not the Drupal query. But has same namingRemoving that for sure.
-
Erik Seifert →
committed 6323c575 on 2.1.x authored by
Rajab Natshah →
Issue #3347975 by Rajab Natshah: Fix Entity queries must explicitly set...
-
Erik Seifert →
committed 6323c575 on 2.1.x authored by
Rajab Natshah →
- Status changed to Fixed
over 1 year ago 5:11pm 20 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.