webfactory / WebfactoryPolyglotBundle

Symfony bundle simplifying translations in Doctrine.
MIT License
3 stars 2 forks source link

Allow `doctrine/persistence:^3.1` #38

Closed relthyg closed 4 months ago

relthyg commented 4 months ago

As this is needed by symfony/doctrine-bridge:7.0.

relthyg commented 4 months ago

Ich habe keine Idee, wie ich den einen Test fixen kann, der failt. Vielleicht haben @mpdude oder @FabianSchmick eine?

mpdude commented 4 months ago

Ich habe das schon mal irgendwo gesehen… 🤔

mpdude commented 4 months ago

Vielleicht war es in der Vorbereitung neuer Features, die insb. über #28, #30 #31 im 3.x Branch gelandet sind… aber der Branch kann auch noch kein Persistence 3.x…

relthyg commented 4 months ago

Die in Version 3.0 von doctrine/persistence eingeführteRuntimeReflectionProperty hat gegenüber der ReflectionProperty noch die zusätzliche Eigenschaft key, welche beim Aufruf von sleep() im Test nicht übertragen wird (und auch nicht übertragen werden kann, weil sie private ist), darum sind die beiden Objekte im TranslatableClassMetadataTest::can_be_serialized_and_retrieved() nicht mehr gleich.

mpdude commented 4 months ago

https://github.com/doctrine/persistence/pull/307 is the reason for those new property types. I think we should use these also after serialization/unserialization of our metadata, so this needs to be investigated.