Trim reflected column names

pauldex / sqlalchemy-firebird

A Firebird dialect for SQLAlchemy using the firebird-driver and/or fdb python Firebird driver

MIT License

23 stars 15 forks source link

Trim reflected column names #2

Closed gordthompson closed 4 years ago

gordthompson commented 4 years ago

get_columns was returning field names with trailing spaces, causing expressions like table_a.c.ID to fail with a KeyError because the field name was 'ID ', not 'ID'. Fixes test failures in "BuggyDomainReflectionTest".

This PR also includes the changes from #1