Skip to content
This repository has been archived by the owner on Feb 24, 2023. It is now read-only.

Fixes support for doctrine/persistence 2.x #691

Merged
merged 1 commit into from
Aug 25, 2020
Merged

Fixes support for doctrine/persistence 2.x #691

merged 1 commit into from
Aug 25, 2020

Conversation

fruitwasp
Copy link
Contributor

No description provided.

@andrea-daru
Copy link

I need this fix to be released because it's blocking for my project.
@fruitwasp do you know when it will be released?

@michaljusiega
Copy link

cc @fabpot - Can you tell when this fix will be released ?

@fabpot
Copy link
Member

fabpot commented Aug 25, 2020

Apparently, the tests are broken, so it cannot be merged as is.

@jhfyang
Copy link

jhfyang commented Aug 25, 2020

Hello is it possible to try to import with :
use Symfony\Bridge\Doctrine\ManagerRegistry; ?

@fabpot
Copy link
Member

fabpot commented Aug 25, 2020

Taking over as Doctrine, once again, broke BC the hard way.

@fabpot
Copy link
Member

fabpot commented Aug 25, 2020

Thank you @fruitwasp.

@fabpot fabpot merged commit 47b0fe0 into sensiolabs:master Aug 25, 2020
@fabpot
Copy link
Member

fabpot commented Aug 25, 2020

This is due to a BC break in Doctrine (see doctrine/DoctrineBundle#1202). Fixed now in v5.6.0. Everything should work again.

fabpot added a commit that referenced this pull request Aug 25, 2020
…ęga)

This PR was squashed before being merged into the 5.6.x-dev branch.

Discussion
----------

Added missing changes for doctrine/persistence.

Not everything has been changed in #691

Commits
-------

2906468 Added missing changes for doctrine/persistence.
artem0723 added a commit to artem0723/React-Clothing-Shop that referenced this pull request Aug 27, 2021
…ęga)

This PR was squashed before being merged into the 5.6.x-dev branch.

Discussion
----------

Added missing changes for doctrine/persistence.

Not everything has been changed in sensiolabs/SensioFrameworkExtraBundle#691

Commits
-------

2906468 Added missing changes for doctrine/persistence.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants