- Issue created by @codebymikey
- @codebymikey opened merge request.
- Status changed to Needs review
over 1 year ago 12:42pm 6 July 2023 - Status changed to Needs work
over 1 year ago 6:23am 11 July 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Thanks for the patch @codebymikey. A few remarks and then it's good to go.
-
+++ b/src/Collector.php @@ -81,7 +81,7 @@ class Collector implements CollectorInterface { + $query = $storage->getQuery()->accessCheck(FALSE);
This line is a duplicate with 87 which was added in 🐛 [error] Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not. Fixed
-
+++ b/status_dashboard.install @@ -95,7 +95,7 @@ function status_dashboard_update_9103() { + $query = $storage->getQuery()->accessCheck(FALSE);
I prefer this to be on a new line like the rest.
-
- Status changed to Needs review
over 1 year ago 9:27am 14 July 2023 -
BramDriesen →
committed 9d40bfef on 2.0.x authored by
codebymikey →
Issue #3372868: Fix access check on query
-
BramDriesen →
committed 9d40bfef on 2.0.x authored by
codebymikey →
- Status changed to Fixed
over 1 year ago 2:27pm 14 July 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Thanks! Will tag a new release in a bit as well.
Automatically closed - issue fixed for 2 weeks with no activity.