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

Use doctrine/orm 3.0 #756

Open
jfsenechal opened this issue Feb 4, 2024 · 1 comment
Open

Use doctrine/orm 3.0 #756

jfsenechal opened this issue Feb 4, 2024 · 1 comment

Comments

@jfsenechal
Copy link

Hi,

Doctrine orm 3.0 was released yesterday, and it's good news :-)
And from today when you install symfony webapp it comes with doctrine 3.0!

So I have already made progress in this implementation :

Replace use Doctrine\ORM\Mapping\ClassMetadataInfo by use Doctrine\ORM\Mapping\ClassMetadata
Remove cascade 'merge'

up in composer

"doctrine/annotations": "^2.0.1"
"doctrine/orm": "^3.0.0"

Ref
https://github.com/doctrine/orm/blob/3.0.x/UPGRADE.md#bc-break-removed-doctrineormmappingclassmetadatainfo-class
https://github.com/doctrine/orm/blob/3.0.x/UPGRADE.md#bc-break-remove-ability-to-merge-detached-entities

you can see my last commits

https://github.com/jfsenechal/DoctrineBehaviors/commits/sf7.0/

I have two failures with blameable

There were 2 failures:

1) Knp\DoctrineBehaviors\Tests\ORM\Blameable\BlameableTest::testCreate
'user' does not match expected type "object".

/var/www/doctrinebehaviors/tests/ORM/Blameable/BlameableTest.php:43

2) Knp\DoctrineBehaviors\Tests\ORM\Blameable\BlameableTest::testExtraSqlCalls
Failed asserting that actual size 3 matches expected size 7.

/var/www/doctrinebehaviors/tests/ORM/Blameable/BlameableTest.php:138
@FluffyDiscord
Copy link

you should keep doctrine 2.19 in the composer.json for now

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