-
|
Hi π , After upgrade from 6.2 to 6.3, my lint container is not working anymore. I got:
I searched in the changelog but i didn't anything related. If you have any hints! Thanks |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 13 replies
-
|
You reported on #50509 (comment) some days ago; didnβt #50585 solve your issue? If not, please provide a reproducer. |
Beta Was this translation helpful? Give feedback.
-
|
Hello, I tried but no, it didnβt solve my issue. Ok thanks, I will try to do a reproducer. |
Beta Was this translation helpful? Give feedback.
-
|
I can see this fix hasn't been released yet. |
Beta Was this translation helpful? Give feedback.
-
|
I tried by fetching the origin branch. |
Beta Was this translation helpful? Give feedback.
-
|
Are you sure? Do you have changes from this PR inside vendor/symfony/cache directory? |
Beta Was this translation helpful? Give feedback.
-
|
Yep I check the logs I had the merged PR, I cleared the cache. |
Beta Was this translation helpful? Give feedback.
-
|
Anyone using Redis as cache can launch |
Beta Was this translation helpful? Give feedback.
-
|
Ok after digging more, i found out that .env.dev: But I don't understand why π€ |
Beta Was this translation helpful? Give feedback.
-
|
I don't understand either. But 6.2 was OK. What's the method for the stack trace? I have a service that create a Redis service: So this way i can quickly inject my Redis, i thought that was the problem at the very first begging so i completely remove it and associated lines in my services files. But it does not change anything. I tell you in case it helps you to figure out... I always clear the cache before launching the command. I just tried to reverse the commit from the merge request you linked but same, it does not work and same error. Thanks again for your help π |
Beta Was this translation helpful? Give feedback.
-
|
stack trace: |
Beta Was this translation helpful? Give feedback.
-
|
I don't know if it helps but if i change these 2 lines in my from to it works |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the stack, but I see you edited your post to remove a repositoryβs configuration and itβs part of the problem, because youβre using the expression language. Youβre rather supposed to extend
Now, from your post above you configured caches to use Redis, and you probably configured Doctrine to use these (is this something you want? This is not the case by default in the development environment). The Redis cache adapter will then be created, but So there indeed is a bug (created #50988), but it results from the combination of two things you can change on your side:
|
Beta Was this translation helpful? Give feedback.
-
|
I thought that was not related so i edited but i shouldn't, sorry! I looked at ServiceEntityRepository when it arrived but i struggled with the declaration, I didn't succeed to manually declare repository classes extending ServiceEntityRepository. I tested without doctrine cache and indeed it's working π On the other hand, i updated my repo declaration to remove expression language and use the doctrine factory like this to test if it's working without the expression language: But not working either, weird? You made a point about doctrine cache in dev, i should not have this behavior, and i will look to find how to switch to ServiceEntityRepository and manually declare it to make my code cleaner. Thanks for all the time you took for me, very appreciated π π |
Beta Was this translation helpful? Give feedback.
Thanks for the stack, but I see you edited your post to remove a repositoryβs configuration and itβs part of the problem, because youβre using the expression language. Youβre rather supposed to extend
ServiceEntityRepository, but letβs continue anyway.lint:containerwill only run one compiler pass: theCheckTypeDeclarationPass, which must evaluate expressions to do its job. Because your expression callsEntityManagerInterface::getClassMetadata, the entity manager has to be instantiated as well as all its arguments, recursively (because it has to be fully functional to be used). Among these arguments is aConfigurationwhich can store some cache implementations.Now, from your post above β¦