- Issue created by @icurk
- 🇸🇮Slovenia icurk
In Drupal 10, accessCheck() method is required in entityQuery calls. Here is a patch that adds the required method to the entityQuery call.
- Status changed to Needs review
almost 2 years ago 2:30pm 14 February 2023 - 🇪🇸Spain alvarodemendoza
Patch from #3 is working. Changing status to reviewed.
- Status changed to RTBC
almost 2 years ago 9:49am 15 February 2023 - 🇸🇮Slovenia icurk
Just uploading a patch where I've removed the comments form my IDE.
- Status changed to Needs review
over 1 year ago 6:22am 28 February 2023 - 🇺🇸United States bogdog400
I ended up patching the method in Drupal core (core/lib/Drupal/Core/Entity/Query/Sql/Query.php) to add a call to accessCheck. This seems to remove this problem. It's not a good solution since they seem to want to push the call to accessCheck out to the modules.
/** * {@inheritdoc} */ public function execute() { return $this ->accessCheck(TRUE) ->prepare() ->compile() ->addSort() ->finish() ->result(); }
Another solution is to patch the call to execute in src/LicenseStorage.php with a call to accessCheck:
public function getExistingLicense(ProductVariationInterface $variation, $uid) {
Also there's a call in src/Cron.php:
protected function getLicensesToExpire(int $time): array {
- Status changed to RTBC
over 1 year ago 4:01pm 4 May 2023 - First commit to issue fork.
- Status changed to Needs review
about 1 year ago 7:53am 6 November 2023 - 🇺🇦Ukraine AstonVictor
As I can see, It was already done on https://www.drupal.org/project/prevnext/issues/3289140 📌 Automated Drupal 10 compatibility fixes Fixed issue but it was committed with
accessCheck = FALSE
which is not correct. - 🇮🇹Italy aronne
Hi guys,
I'm gonna upload a new patch that should solve the problem with the accessCheck call. - Status changed to Fixed
about 1 year ago 8:55am 8 November 2023 - 🇮🇹Italy aronne
Thank you for your contribution, it will get released soon.
Automatically closed - issue fixed for 2 weeks with no activity.