search

zim32 / mysql.dart

MySQL client for Dart written in Dart
BSD 3-Clause "New" or "Revised" License
67 stars 18 forks source link

column matching is case-sensitive #22

Closed bsutton closed 2 years ago

bsutton commented 2 years ago

Just found a problem with the column matching in that this project does a case sensitive match on mysql column names.

Column names in mysql are case insensitive (where as table names are 'normally' case sensitive - depends on platform and settings).

This just caused our project to break as we changed the third party tool that generates our mysql database. The result is that calls to methods such as ResultSetRow.colByName is now failing.

Can I suggest that we push the column defs to lower case and then any column name argument is also pushed to lowercase before we do the match.

zim32 commented 2 years ago

Yes, this is a bug. I will fix it

bsutton commented 2 years ago

Thanks.

On Thu, 26 May 2022, 5:07 pm zim32, @.***> wrote:

Yes, this is a bug. I will fix it

— Reply to this email directly, view it on GitHub https://github.com/zim32/mysql.dart/issues/22#issuecomment-1138232973, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG32OFH4AMV7S34KID6FUTVL4PKXANCNFSM5W7ZTPCQ . You are receiving this because you authored the thread.Message ID: @.***>

zim32 commented 2 years ago

Fixed in 0.0.24