Commit 1dd5bd1
committed
bug symfony#61232 [Messenger] Firebird Database - incompatibility with expected lowercase columns (DSsatwareAG)
This PR was squashed before being merged into the 6.4 branch.
Discussion
----------
[Messenger] Firebird Database - incompatibility with expected lowercase columns
Firebird Database - incompatibility with expected lowercase columns
| Q | A
| ------------- | ---
| Branch? | 6.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | Fix symfony#61171
| License | MIT
Fix ensures, for Firebird databases, the column names are consistently converted to lowercase before being processed by Doctrine Messenger. This resolves the incompatibility by providing Messenger with the expected lowercase column names, regardless of how Firebird returns them.
Because the FirebirdPlatform is not part of the doctrine/dbal package, I was unable to use instanceof for platform detection. Instead, I implemented a string comparison to check for Firebird as the database platform.
Commits
-------
6223362 [Messenger] Firebird Database - incompatibility with expected lowercase columnsFile tree
1 file changed
+4
-1
lines changed- src/Symfony/Component/Messenger/Bridge/Doctrine/Transport
1 file changed
+4
-1
lines changedLines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| 36 | + | |
36 | 37 | | |
37 | 38 | | |
38 | 39 | | |
| |||
405 | 406 | | |
406 | 407 | | |
407 | 408 | | |
408 | | - | |
| 409 | + | |
| 410 | + | |
| 411 | + | |
409 | 412 | | |
410 | 413 | | |
411 | 414 | | |
| |||
0 commit comments