-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[Messenger Doctrine] Fixed regression by #50524 causing data loss #50716
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
Closed
Closed
Changes from 1 commit
Commits
Show all changes
4808 commits
Select commit
Hold shift + click to select a range
3fcf300
bug #50307 [AssetMapper] Improving XSD to use attributes whenever pos…
nicolas-grekas 657e9be
[Notifier] Bring consistency to bridges
nicolas-grekas e3a140d
[Security] Skip clearing CSRF Token on stateless logout
chalasr ce77ed8
[AssetMapper] Add cached asset factory
weaverryan 3514acd
feature #50286 [AssetMapper] Add cached asset factory (weaverryan)
fabpot e7f5216
Update CHANGELOG for 6.3.0-BETA3
fabpot d93cf13
Update VERSION for 6.3.0-BETA3
fabpot ddb0546
Merge pull request #50313 from fabpot/release-6.3.0-BETA3
fabpot 63f1f71
Bump Symfony version to 6.3.0
fabpot 670e0f3
Merge branch '5.4' into 6.2
nicolas-grekas 3471e34
Merge branch '6.2' into 6.3
nicolas-grekas 8c637a5
minor #50308 [Notifier] Bring consistency to bridges (nicolas-grekas)
nicolas-grekas 752378c
Merge branch '5.4' into 6.2
nicolas-grekas eb3e308
Merge branch '6.2' into 6.3
nicolas-grekas 38021b5
[Console] Remove redundant method getSaturation()
ivanpepelko df086fa
minor #50317 [Console] Remove redundant method getSaturation() (ivanp…
fabpot c9ed471
[WebProfilerBundle] Tweak the HTML code of the Twig entry view
javiereguiluz c036acf
Ensure DoctrineIntegrationTest always run in UTC
alex-dev 398830c
minor #50328 Ensure DoctrineIntegrationTest always run in UTC (alex-dev)
nicolas-grekas 00df479
Provide a value if possible when parameter is not found
MatTheCat 6b0ebc8
[VarDumper] Fix HTML of invisible characters
fancyweb 796a65b
bug #50343 [VarDumper] Fix HTML of invisible characters (fancyweb)
nicolas-grekas b7b0c97
bug #50340 [HttpKernel] Make `QueryParameterValueResolver` provide a …
nicolas-grekas 39bb6b6
Fix problem with empty generator in StreamedJsonResponse
alexander-schranz 8d24399
bug #50331 [HttpFoundation] Fix problem with empty generator in Strea…
nicolas-grekas 5c40d30
bug #50325 [WebProfilerBundle] Tweak the HTML code of the Twig entry …
nicolas-grekas 53046a3
[VarDumper] Reduce stylesheet assignments via JavaScript in `HtmlDumper`
ohader 825fd03
minor #49977 [VarDumper] Reduce stylesheet assignments via JavaScript…
nicolas-grekas 09c7581
[DependencyInjection] Add exclude-self option to dumpers
HypeMC d3486dd
Fix get sender name in turbosms notifier
ZhukV 3d6e0bd
bug #50351 [DependencyInjection] Add `excludeSelf` option to dumpers …
nicolas-grekas bf262b9
bug #50352 [Notifier][TurboSMS] Fix get sender name (ZhukV)
nicolas-grekas 51b9246
Bug fix for paths that start with the same string
weaverryan 8d7ac4c
bug #50355 Bug fix for paths that start with the same string (weaverr…
nicolas-grekas 749b304
Merge branch '5.4' into 6.2
nicolas-grekas a689085
bug #50280 [PropertyAccess] Fix nullsafe operator on array index (Hyp…
nicolas-grekas 240c031
[Serializer] Fix SerializedPath not working with constructor arguments
HypeMC 71f8c57
bug #50193 [Serializer] Fix `SerializedPath` not working with constru…
nicolas-grekas eab290d
Merge branch '6.2' into 6.3
nicolas-grekas 0e61a66
[DoctrineBridge] Allow to ignore specific nullable fields in UniqueEn…
VincentLanglet 95aa09e
feature #49293 [DoctrineBridge] Allow to ignore specific nullable fie…
nicolas-grekas aef515f
[Validator] Allow to use translation_domain false for validators and …
VincentLanglet 92e213e
feature #48852 [Validator] Allow to use translation_domain false for …
nicolas-grekas 93cc075
[Security] Improve DX when invalid custom authenticators
alamirault cf5f103
minor #49938 [Security] Improve DX when invalid custom authenticators…
nicolas-grekas a3a5472
[AssetMapper] Adding "path" option to importmap:require
weaverryan 8355439
feature #50363 [AssetMapper] Adding "path" option to importmap:requir…
nicolas-grekas 8a8b5cd
[VarDumper][DebugBundle] Fix dump labels compatibility
fancyweb 761d915
bug #50347 [DebugBundle][VarDumper] Fix dump labels compatibility (fa…
nicolas-grekas d0245f8
[AssetMapper] Fixing improper use of hasOption() in command
weaverryan 398036b
minor #50365 [AssetMapper] Fixing improper use of hasOption() in comm…
nicolas-grekas feb680b
[AssetMapper] Fix bug where dependencies were preloaded even if the p…
weaverryan 54a3b84
bug #50356 [AssetMapper] Fix bug where dependencies were preloaded ev…
nicolas-grekas 5b92940
Merge branch '5.4' into 6.2
nicolas-grekas 2c33e56
Fix merge
nicolas-grekas 684fdd2
bug #50312 [Security] Skip clearing CSRF Token on stateless logout (c…
nicolas-grekas 6d97baa
Merge branch '6.2' into 6.3
nicolas-grekas 34c8f0c
Fix merge
nicolas-grekas 099ba75
[Security] Test `CsrfTokenClearingLogoutListener` with stateless logout
chalasr 457d56f
minor #50369 [Security] Test `CsrfTokenClearingLogoutListener` with s…
chalasr 6153f04
[Notifier] Document Notifier options in README files
alamirault 746c06b
bug #50349 [Notifier] Document Notifier options in README files (alam…
nicolas-grekas 9371536
fix exception wording
garak 4a4493c
minor #50381 [HttpKernel] fix exception wording (garak)
wouterj ebcced6
[Scheduler] Improve triggers performance when possible
fabpot e770344
[Notifier] Document Notifier options in README files
alamirault a3124b7
minor #50378 [Notifier] Document Notifier options in README files (al…
nicolas-grekas c041d91
[WebProfilerBundle] Fix cursor on link that has no href
PhilETaylor 3f9fe23
bug #50376 [WebProfilerBundle] Fix cursor on link that has no href (P…
nicolas-grekas 9b0d811
bug #49817 [Scheduler] Improve triggers performance when possible (fa…
nicolas-grekas 656749f
Merge branch '5.4' into 6.2
nicolas-grekas 8e7ea19
Merge branch '6.2' into 6.3
nicolas-grekas 893d299
Convert A link to Button
PhilETaylor 6a8e73d
Update CHANGELOG for 6.3.0-RC1
fabpot 2f1d3f0
Update VERSION for 6.3.0-RC1
fabpot adb5a03
Merge pull request #50388 from fabpot/release-6.3.0-RC1
fabpot 2979e1e
Bump Symfony version to 6.3.0
fabpot 9f52bf1
[AssetMapper] Fixing incorrect exception & adding allowing more reali…
weaverryan 343b6d7
bug #50393 [AssetMapper] Fixing incorrect exception & adding allowing…
nicolas-grekas 11b8603
CS fix
nicolas-grekas 5bf5c4d
minor #50386 Fix cursor on link that has no href (PhilETaylor)
nicolas-grekas c05fdb8
[AssetMapper] Sometimes asset contents are built from non-asset files
weaverryan 5320c85
Merge branch '5.4' into 6.2
nicolas-grekas 8fd49c1
Merge branch '6.2' into 6.3
nicolas-grekas 07d318a
[VarDumper] Fix `dd()` showing line with `null`
HypeMC a2f2f82
bug #50406 [VarDumper] Fix `dd()` showing line with `null` (HypeMC)
nicolas-grekas 03d8302
bug #50400 [AssetMapper] Sometimes asset contents are built from non-…
nicolas-grekas bb97591
[AssetMapper] Avoid loading potentially ALL assets in dev server
weaverryan cc65825
bug #50394 [AssetMapper] Avoid loading potentially ALL assets in dev …
nicolas-grekas 23c6a35
Update github/workflows/scorecard
nicolas-grekas c9425a2
Merge branch '5.4' into 6.2
nicolas-grekas 27154d0
Merge branch '6.2' into 6.3
nicolas-grekas de44614
[AssetMapper] Change default importmap "provider" to JsDelivr+esm
weaverryan b530dc3
[AssetMapper] Fix wiring resolvers, send requests in parallel and use…
nicolas-grekas f64e38d
bug #50408 [AssetMapper] Change default importmap "provider" to JsDel…
nicolas-grekas 743aff9
Fix: Typo in PHPDoc
OskarStark 52f45a4
[VarDumper] Fix failures on appveyor
nicolas-grekas 066f6b3
minor #50424 [Messenger] Fix: Typo in PHPDoc (OskarStark)
fabpot e824eb0
[VarDumper][HttpKernel] Fix dumping with labels
nicolas-grekas 2f3e24a
bug #50397 [HttpKernel][VarDumper] Fix dumping with labels (nicolas-g…
nicolas-grekas 100f9ea
minor #50427 [VarDumper] Fix failures on appveyor (nicolas-grekas)
nicolas-grekas 4347520
Merge branch '5.4' into 6.2
nicolas-grekas 268e740
Merge branch '6.2' into 6.3
nicolas-grekas 9f31576
[AssetMapper] Using a separate logger channel
weaverryan f65b668
minor #50435 [AssetMapper] Using a separate logger channel (weaverryan)
fabpot d6ea474
[DependencyInjection] Revert "Use weak references in the container"
nicolas-grekas 09a08cb
[HttpKernel] conflict with VarDumper < 6.3
nicolas-grekas 6927654
minor #50443 [HttpKernel] conflict with VarDumper < 6.3 (nicolas-grekas)
nicolas-grekas 542f039
bug #50440 [DependencyInjection] Revert "Use weak references in the c…
nicolas-grekas 7021964
Merge branch '5.4' into 6.2
nicolas-grekas d50daee
Merge branch '6.2' into 6.3
nicolas-grekas ed2641a
[AssetMapper] Adding documentation to importmap.php to guide the user
weaverryan 68f8ad9
minor #50436 [AssetMapper] Adding documentation to importmap.php to g…
fabpot 27ecdb4
[AssetMapper] Adding an "alias" syntax to importmap:require
weaverryan abc5e15
[SecurityBundle] Update security-1.0.xsd to include missing access-to…
aegypius 9608af3
bug #50442 [SecurityBundle] Update security-1.0.xsd to include missin…
fabpot ec6338f
feature #50445 [AssetMapper] Add "=alias" syntax to importmap:require…
fabpot 568b653
Merge branch '6.2' into 6.3
fabpot 50ff60f
Update CHANGELOG for 6.2.11
fabpot 3ce0d07
Update VERSION for 6.2.11
fabpot 626f53a
Merge pull request #50454 from fabpot/release-6.2.11
fabpot 4b72b61
Bump Symfony version to 6.2.12
fabpot 94f492e
Update CHANGELOG for 6.3.0-RC2
fabpot 60588cc
Update VERSION for 6.3.0-RC2
fabpot e885d53
Merge pull request #50455 from fabpot/release-6.3.0-RC2
fabpot e1bf69c
Bump Symfony version to 6.3.0
fabpot 2c668ba
Ignore definitions bearing the `container.excluded` tag
Myks92 4f2b5e1
bug #50452 Hide definitions bearing the `container.excluded` tag (Myk…
chalasr 3ee8d38
Merge branch '5.4' into 6.2
nicolas-grekas acac306
Merge branch '6.2' into 6.3
nicolas-grekas 0a0a2c9
Fix unable to use asset mapper with CSP
vtsykun fabe7bc
[HttpKernel] Fix default value ignored with pinned resolvers
HypeMC 2af8f39
bug #50458 [HttpKernel] Fix default value ignored with pinned resolve…
nicolas-grekas 5bab536
bug #50456 [AssetMapper] Fix unable to use asset mapper with CSP (vts…
fabpot cbe4808
[FrameworkBundle][PhpUnitBridge] Configure doctrine/deprecations as e…
nicolas-grekas 380e709
CS fix
nicolas-grekas 8dd9d93
[Notifier] Fix ContactEveryoneOptions
nicolas-grekas 23c9e17
[SecurityBundle] Fix configuring OIDC user info token handler client
vincentchalamon 5c46e7e
bug #50470 [SecurityBundle] Fix configuring OIDC user info token hand…
nicolas-grekas f2df30b
bug #50473 [Notifier] Fix ContactEveryoneOptions (nicolas-grekas)
nicolas-grekas c96a390
bug #50468 [FrameworkBundle][PhpUnitBridge] Configure doctrine/deprec…
nicolas-grekas 73e3cff
Merge branch '6.2' into 6.3
nicolas-grekas cf78f12
Fix tests
nicolas-grekas 7776c28
[FrameworkBundle] remove support for preloading ESM modules using hea…
dunglas bff0601
[SecurityBundle] add missing xsd definition for OIDC
aegypius b587099
bug #50453 [SecurityBundle] add missing xsd definition for OIDC (aegy…
nicolas-grekas dec93e9
bug #50476 [FrameworkBundle] remove support for preloading ESM using …
nicolas-grekas 5394130
[Security] Add clock dependency to OidcTokenHandler
nicolas-grekas 9a3b651
Merge branch '5.4' into 6.2
nicolas-grekas da82f53
[Serializer] Fix discriminator map not working with `AbstractNormaliz…
HypeMC 875a729
Merge branch '6.2' into 6.3
nicolas-grekas 2654e7b
bug #50477 [Security] Add clock dependency to OidcTokenHandler (nicol…
nicolas-grekas 90011f5
[Security] Validate `aud` and `iss` claims on OidcTokenHandler
vincentchalamon 744c378
bug #50432 [Security] Validate `aud` and `iss` claims on OidcTokenHan…
nicolas-grekas e9afaa3
Update CHANGELOG for 6.3.0
fabpot 9241f84
Update VERSION for 6.3.0
fabpot 85182e7
Merge pull request #50482 from fabpot/release-6.3.0
fabpot 1a9ac8d
Bump Symfony version to 6.3.1
fabpot 91e6e57
[VarDumper] Use documentElement instead of body for JS flag
ohader c31276b
bug #50493 [VarDumper] Use documentElement instead of body for JS fla…
nicolas-grekas e901262
bug #50480 [Serializer] Fix discriminator map not working with `Abstr…
nicolas-grekas 06ab7ec
[FrameworkBundle] ease migration to symfony 6.3
lyrixx 9ae8b6c
add @throws to getPayload
wiseguy1394 c106c5f
[SecurityBundle] Fix error message when using OIDC and web-token/jwt-…
nicolas-grekas 14b1088
bug #50498 [FrameworkBundle] ease migration to symfony 6.3 (lyrixx)
nicolas-grekas 99e1b1f
bug #50503 [SecurityBundle] Fix error message when using OIDC and web…
nicolas-grekas 7e7adcc
minor #50492 [HttpFoundation] add `@throws` to getPayload (wiseguy1394)
nicolas-grekas 02909a2
[Messenger] Add deprecation message for the "messenger.listener.stop_…
alexandre-daubois 2e688f7
bug #50508 [Messenger] Add deprecation message for the `messenger.lis…
nicolas-grekas 03392b6
[PhpUnitBridge] Disable deduplication of Doctrine deprecations
nicolas-grekas 5ffa61c
[SecurityBundle] Use LogicException instead of Definition::addError()
nicolas-grekas 2a29674
[VarDumper] Add named arguments coverage of VarDumper's function
alexandre-daubois 96c2dd7
minor #50512 [VarDumper] Add named arguments coverage of VarDumper's …
nicolas-grekas 29fe9ff
bug #50514 [PhpUnitBridge] Disable deduplication of Doctrine deprecat…
nicolas-grekas b033925
minor #50519 [SecurityBundle] Use LogicException instead of Definiti…
nicolas-grekas 38a7ee5
[Mailer] [MailPace] Fix undefined array key in errors response
flofloflo a88e5f8
bug #50515 [Mailer] [MailPace] Fix undefined array key in errors resp…
nicolas-grekas 24347bf
Merge branch '5.4' into 6.2
nicolas-grekas e5c5a51
Merge branch '6.2' into 6.3
nicolas-grekas a0a3ce1
[Serializer] Fix ignoring objects that only implement DenormalizableI…
7e01a32
bug #50521 [Serializer] Fix ignoring objects that only implement Deno…
nicolas-grekas c7fb0fd
[PhpUnitBridge] Add missing line in changelog
nicolas-grekas d2b5209
[PhpUnitBridge] Fix classifying doctrine/deprecations as direct/indirect
nicolas-grekas 9217c57
bug #50525 [PhpUnitBridge] Fix classifying doctrine/deprecations as d…
nicolas-grekas 5101d18
[DependencyInjection] Fix support for `false` boolean env vars
okhoshi 538a9ab
[Messenger] Prevent `StopWorkerOnSignalsListener::$signals` to be ass…
alexandre-daubois 26891af
bug #50532 [Messenger] Prevent `StopWorkerOnSignalsListener::$signals…
fabpot 5b6573c
[PhpUnitBridge] Fix requiring bootstrap.php
nicolas-grekas f3b6e77
Merge branch '5.4' into 6.2
nicolas-grekas 3fd32aa
fix merge
xabbuh a859c40
fix Doctrine deprecations
xabbuh 0665a3b
minor #50544 fix Doctrine deprecations (xabbuh)
fabpot a2498b8
Merge branch '6.2' into 6.3
xabbuh e41e47a
Merge branch '5.4' into 6.2
xabbuh 061d5db
Merge branch '6.2' into 6.3
xabbuh 330bb6a
[DoctrineBridge] Update deprecations baseline
nicolas-grekas 4915f12
minor #50549 [DoctrineBridge] Update deprecations baseline (nicolas-g…
nicolas-grekas bb0684a
Merge branch '5.4' into 6.2
nicolas-grekas 8f9cc4c
Merge branch '6.2' into 6.3
nicolas-grekas 620e399
Ignore doctrine deprecations
nicolas-grekas d16b931
Merge branch '5.4' into 6.2
nicolas-grekas 2699886
Merge branch '6.2' into 6.3
nicolas-grekas e4c90a3
-
nicolas-grekas 2c71034
-
nicolas-grekas 31d2b01
-
nicolas-grekas 7c7b0b2
GH-50526: Reverting ExecutionContextInterface void return types in fa…
upchuk 1cceec1
bug #50540 [Validator] GH-50526: Reverting ExecutionContextInterface …
nicolas-grekas 1e22c3b
remove unusable cache pools
xabbuh f6efec5
[SecurityBundle] Mark SignatureAlgorithmFactory as internal
fancyweb ac88b34
minor #50564 [SecurityBundle] Mark SignatureAlgorithmFactory as inter…
nicolas-grekas ce09da1
[PhpUnitBridge] Ignore deprecations about the annotation mapping driv…
nicolas-grekas 1693989
bug #50567 [PhpUnitBridge] Ignore deprecations about the annotation m…
nicolas-grekas db20408
bug #50563 [FrameworkBundle] remove unusable cache pools (xabbuh)
nicolas-grekas 77521de
Fix .github/expected-missing-return-types.diff
nicolas-grekas 28c61f2
Merge branch '5.4' into 6.2
nicolas-grekas eeb92df
Merge branch '6.2' into 6.3
nicolas-grekas 29581c8
Merge branch '5.4' into 6.2
nicolas-grekas c973874
Merge branch '6.2' into 6.3
nicolas-grekas d65db40
[DoctrineBridge] Remove (wrong) PHPDoc on `ContainerAwareEventManager`
alexandre-daubois 40a8561
widen return type for Monolog 3 compatibility
xabbuh 5e06cb3
Merge branch '5.4' into 6.2
derrabus 40023eb
Merge branch '6.2' into 6.3
derrabus d63de1b
Merge branch '5.4' into 6.2
nicolas-grekas 9f9edf6
Merge branch '6.2' into 6.3
nicolas-grekas 26147ce
minor #50594 [DoctrineBridge] Remove (wrong) PHPDoc on `ContainerAwar…
nicolas-grekas f6c8b0f
bug #50599 [MonologBridge] widen return type for Monolog 3 compatibil…
nicolas-grekas 6f72d4f
[Notifier] Add docs about updating Slack messages
javiereguiluz b1bd15d
minor #50581 [Notifier] Add docs about updating Slack messages (javie…
nicolas-grekas 4df982d
Merge branch '5.4' into 6.2
nicolas-grekas 8ffdd91
Merge branch '6.2' into 6.3
nicolas-grekas b66e6db
Merge branch '5.4' into 6.2
nicolas-grekas b16b1dc
Merge branch '6.2' into 6.3
nicolas-grekas cf93c82
Merge branch '5.4' into 6.2
nicolas-grekas 77106b9
Merge branch '6.2' into 6.3
nicolas-grekas e6e1e26
Merge branch '5.4' into 6.2
nicolas-grekas 0320b68
Merge branch '6.2' into 6.3
nicolas-grekas 8881fac
[GHA] Skip loading ext-relay when it's not available
nicolas-grekas c3f9b41
minor #50610 [GHA] Skip loading ext-relay when it's not available (ni…
nicolas-grekas 7361374
[Clock] Fix MockClock::modify() on PHP 8.3
nicolas-grekas eacea7f
bug #50611 [Clock] Fix MockClock::modify() on PHP 8.3 (nicolas-grekas)
nicolas-grekas 2f9301c
Merge branch '6.2' into 6.3
nicolas-grekas fbd6d18
[HttpClient] Remove final keyword on AsyncResponse
lyrixx d9a8902
bug #50577 [HttpClient] Remove final keyword on `AsyncResponse` (lyrixx)
nicolas-grekas 87a65d8
bug #50530 [DependencyInjection] Fix support for `false` boolean env …
nicolas-grekas 5b66f26
CS fix
nicolas-grekas 1209b81
Don't mark RedispatchMessage as internal
valtzu 5f29bd8
[Security] Fix log message in OidcTokenHandler
chalasr 12d668c
bug #50686 [Messenger] Don't mark `RedispatchMessage` as internal (va…
fabpot df2ae1e
minor #50696 [Security] Fix log message in `OidcTokenHandler` (chalasr)
fabpot 8d790d8
[Messenger Doctrine] Fixed regression by #50524 causing data loss
brusch File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev
Previous commit
[Messenger Doctrine] Fixed regression by #50524 causing data loss
When using `$platform->getAlterSchemaSQL($schemaDiff)` instead of `$schemaDiff->toSaveSql($platform)` causes to call `\Doctrine\DBAL\Schema\SchemaDiff::_toSql($platform, false)` instead of `\Doctrine\DBAL\Schema\SchemaDiff::_toSql($platform, true)`. When $saveMode=false the diff from the schema is getting remove, so actually all other tables ... in the DB are getting deleted
- Loading branch information
commit 8d790d852bad7a420f5edd6854a9df46d4f3dad0
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But toSaveSql is deprecated, isn't it? so we need to find the future-proof version of this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's true, but I haven't found an adequate replacement yet 🥲 Therefore I'd keep the current implementation until there's a solution offered by Doctrine.
Probably
doctrine/dbal
needs to add a parameter togetAlterSchemaSQL()
or offers an additional method for that 🤔