- Issue created by @baskaran
- Status changed to Needs review
6 months ago 6:03am 5 August 2024 - Status changed to RTBC
6 months ago 8:03am 5 August 2024 - 🇮🇳India riddhi.addweb
I applied the patch and it applied cleanly, also the issue is resolved it shows some warnings also. Moving it into RTBC. Attaching SS for the same.
- Status changed to Needs work
5 months ago 7:58pm 29 August 2024 - Status changed to Needs review
4 months ago 7:50am 16 September 2024 - Status changed to RTBC
4 months ago 6:59am 18 September 2024 - 🇮🇳India riddhi.addweb
I applied the patch and it applied cleanly, also the issue is resolved it shows some warnings also. Moving it into RTBC. Attaching SS for the same.
- 🇺🇸United States jeanwang2dev
Patch does not work for 9.10-alpha1, It has code from before that is already in the branch.
I edited the patch, remove the old code, this works for me. - Status changed to Needs review
about 1 month ago 8:53pm 17 December 2024 - 🇷🇺Russia yuraosn Rostov-on-don
#8 Thank you.
But you also need to change the menu_token.info.yml. It worked for me. - 🇷🇺Russia yuraosn Rostov-on-don
1 more patch is needed. He corrects this error.
[error] Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not. See Drupal\Core\Entity\Query\QueryInterface::accessCheck(). in Drupal\Core\Entity\Query\Sql\Query->prepare() (line 142 of /var/www/mr-gruppe.de/web/core/lib/Drupal/Core/Entity/Query/Sql/Query.php). - 🇫🇷France tostinni
@yuraosn when you add a fix, you need to make a patch with also your previous fixes so we can test only one patch.
Here is a patch merging patches from #8, #9 and #10.Also it seems that a lot of work has been done in the related MR by @baskaran (see commit c1d7b5ef) but it hasn't been pushed, shouldn't we start from there ?
Finally there is some duplicated effort going on in 🐛 Drupal 11 Compatibility Active which include compatibility with Drupal 11, so I guess we should focus on one issue only.