-
Notifications
You must be signed in to change notification settings - Fork 59
Support for new TokenType type alongside with compatibility for older doctrine/orm versions #76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…der doctrine versions
@jsor Any updates for this? |
@jsor same question - any news? Is critical for me :( |
@imancos-xsoft I ended up working with my fork. I can make it public on packagist.org if you want so you can use it |
@MathisBurger Hey, sorry for being unresponsive for so long. I've added you as an collaborator to this repo. So, feel free to merge and release. |
@jsor @MathisBurger It would be better to merge this PR #61 which includes these changes. |
@jsor No problem. @maxhelias sounds good to me. Should I create then create a new release just after merging #61 ? |
Sure, go ahead 👍 |
I have introduced an alias class that dynamically selects either Doctrine\ORM\Query\Lexer or Doctrine\ORM\Query\TokenType for token constants. This approach allows compatibility with a wide range of doctrine/orm versions while using the same version of jsor. As a result, we support both newer and older versions of Doctrine seamlessly.
I also had to edit psalm and test creation in order to support latest versions of doctrine