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

Fixing deprecation #518

Merged
merged 1 commit into from
Mar 27, 2020
Merged

Fixing deprecation #518

merged 1 commit into from
Mar 27, 2020

Conversation

nikophil
Copy link
Contributor

Hi,

just fixing a deprecation from doctrineDoctrine\Common\Persistence to Doctrine\Persistence

@nikophil
Copy link
Contributor Author

Actually the problem was a little more tricky: we need to fix the deprecation to have the CI green, but we still need to support the version the bundles pretends to support. So, the object \Symfony\Bundle\MakerBundle\Doctrine\DoctrineHelper needs to work with both new and deprecated dependencies

@LeJeanbono
Copy link
Contributor

Hi @nikophil , can you had Doctrine\Common\Persistence\ObjectManager please ?
From #528
Thanks

@weaverryan
Copy link
Member

Sorry for the delay on merging this! Thanks @nikophil! I think there may be at least one other spot in the code (maybe more) that need to be update, but let's merge this and then take a look.

@weaverryan weaverryan merged commit 2600f5a into symfony:master Mar 27, 2020
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

Successfully merging this pull request may close these issues.

3 participants