Skip to content
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

Upgrade ecotone/dbal to use latest doctrine/persistence #111

Closed
xNarkon opened this issue May 3, 2023 · 6 comments
Closed

Upgrade ecotone/dbal to use latest doctrine/persistence #111

xNarkon opened this issue May 3, 2023 · 6 comments

Comments

@xNarkon
Copy link

xNarkon commented May 3, 2023

Description
Currently on Symfony 6.2 there is an issue related to the doctrine/persistence. Symfony recommends to use the doctrine/persistence in the latest version 3+ to use of lazy ghost objects. Of course we can disable this feature for now to avoid the problems using a switch: enable_lazy_ghost_objects but it would be really nice to just upgrade dependencies in ecotone/dbal.

@dgafka
Copy link
Member

dgafka commented May 4, 2023

Hello @xNarkon

It's related to enqueue/dbal, it was not updated to persistence version >= 3.0.
Are you willing to help with this and provide PR in their dev-repo that will bump the version?

@xNarkon
Copy link
Author

xNarkon commented May 4, 2023

@dgafka well, it's already in progress php-enqueue/enqueue-dev#1290. I decided to add my own PR to allow 3.1.

php-enqueue/enqueue-dev#1300 could you push it a bit, as I can see you was a contributor?

Thanks in advance!

@dgafka
Copy link
Member

dgafka commented May 5, 2023

Thanks @jlabedo for updating that :)

@xNarkon
Copy link
Author

xNarkon commented May 12, 2023

@dgafka php-enqueue has already my changes merge

@dgafka
Copy link
Member

dgafka commented May 12, 2023

cool @xNarkon, I think we don't need to update any package constraint on Ecotone side, right?

@dgafka
Copy link
Member

dgafka commented May 16, 2023

Closing as library was upgraded :)

@dgafka dgafka closed this as completed May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants