Closed fabpot closed 5 months ago
Q | A |
---|---|
License | MIT |
Doc issue/PR | symfony/symfony-docs#... |
Thanks for the PR 😍
Define the SYMFONY_ENDPOINT
environment variable:
# On Unix-like (BSD, Linux and macOS)
export SYMFONY_ENDPOINT=https://raw.githubusercontent.com/symfony/recipes/flex/pull-1290/index.json
# On Windows
SET SYMFONY_ENDPOINT=https://raw.githubusercontent.com/symfony/recipes/flex/pull-1290/index.json
Install the package(s) related to this recipe:
composer req 'symfony/flex:^1.16'
composer req 'doctrine/doctrine-bundle:^2.10'
Don't forget to unset the SYMFONY_ENDPOINT
environment variable when done:
# On Unix-like (BSD, Linux and macOS)
unset SYMFONY_ENDPOINT
# On Windows
SET SYMFONY_ENDPOINT=
In order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. I'm going keep this comment up to date with any updates of the attached patch.
@derrabus @fabpot this will however lead to deprecations on DBAL 4. See https://github.com/symfony/demo/pull/1494#issuecomment-1945549526
:confused:
Can you open an issue for that in the DoctrineBundle repository? We should fix it there.
Can you open an issue for that in the DoctrineBundle repository? We should fix it there.
Hm but how would you fix it on DoctrineBundle? I mean on DBAL 4 not setting use_savepoints
at all seems the way to go, or? This does not cause any deprecation. Eventually we should deprecate and remove the use_savepoints
option from the Bundle I guess.
Hm but how would you fix it on DoctrineBundle?
use_savepoints: true
should be a no-op on DBAL 4 while use_savepoints: false
should error.
I mean on DBAL 4 not setting
use_savepoints
at all seems the way to go, or?
Yes. But projects should be able to gradually upgrade which is why it should be fine to still define this option on DBAL 4, even if there's only one valid setting.