Closed zzzeek closed 6 years ago
Original comment by Anonymous:
Now this ticket is resolved by #155 and can be closed. Reflection of mixed case objects works adding natural_case=False when creating Table instances.
Original comment by Anonymous:
forgot remove one ".upper()"
Original comment by Anonymous:
post rev 1799 patch to firebird reflection
Original comment by Anonymous:
Latest svn updates make a great job toward the solution, but there is need in firebird.py to remove some table.name.upper() to make it work. In attached patch I try also to detect if quote=True must be added when creating reflected columns.
Original comment by Michael Bayer (Bitbucket: zzzeek, GitHub: zzzeek):
if i understand this bug correctly i think #155 should actually resolve it; as if your identifier requires quoting, you dont put the quotes in the identifier name itself, you say quote=True
in your Table
and quoting will be applied as appropriate; reflection is therefore not affected by the need for quoting.
feel free to reopen this ticket if problems persist
Originally reported by: Anonymous
Hi, I'm completely new on SQlAlchemy, I'm investigating if SQLAlchemy can be useful for my works. I make large use of firebird, and first thing I checked is the ability to reuse my existing firebird databases. There a problem in current trunk revision with quoted identifiers. Seems reflecttable() works only with unquoted table and fields.
I have done a little patch that let me able to do reflection. Plese don't considerer this as a working patch, I'm new to SQlAlchemy and have much to learn, but can be useful if some guy will make next revision compatible with mixed case quoted identifiers.
Ezio