[Messenger] Firebird Database - incompatibility with expected lowercase columns #61232
Conversation
DSsatwareAG
changed the title
OskarStark
changed the title
carsonbot
changed the title
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
Outdated
Show resolved
Hide resolved
|
Are you using the connection for the messenger exclusively? If so, you could use DBAL's portability middleware to control the casing of result columns. |
Why would you be unable to use |
You’re right — I can use instanceof. My initial concern was a potential fatal if the class wasn’t available. I will updated the PR. |
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
Outdated
Show resolved
Hide resolved
nicolas-grekas
left a comment
nicolas-grekas
left a comment
There was a problem hiding this comment.
@derrabus wrote this:
Are you using the connection for the messenger exclusively? If so, you could use DBAL's portability middleware to control the casing of result columns.
WDYT of this proposal? Applicable?
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
Outdated
Show resolved
Hide resolved
Unfortunately not applicable here — the Doctrine connection is shared, not Messenger‑exclusive. |
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
Outdated
Show resolved
Hide resolved
fabpot
force-pushed
the
doctrine-messenger-firebird-uppercase-columns
branch
from
b2bdbc8 to
6223362
Compare
|
Thank you @DSsatwareAG. |
DSsatwareAG
deleted the
doctrine-messenger-firebird-uppercase-columns
branch
6 participants
Firebird Database - incompatibility with expected lowercase columns
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.