Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix numeric compare affinity for SQLite #989

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Conversation

mvorisek
Copy link
Member

@mvorisek mvorisek commented May 2, 2022

fix #950

in theory, the fix should be done on whole SQL AST, but such approach requires AST parser, so for simplicity, the fix is done on comparison expressions created using Query::where() method only

@mvorisek mvorisek added the bug label May 2, 2022
@mvorisek mvorisek changed the title Fix float value bind for Sqlite Fix numerical compare for Sqlite May 2, 2022
@mvorisek mvorisek changed the title Fix numerical compare for Sqlite Fix numeric compare for Sqlite May 2, 2022
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 10 times, most recently from f3bc2d8 to c7b3b9d Compare May 6, 2022 22:10
@mvorisek mvorisek added the MAJOR label May 6, 2022
@mvorisek mvorisek changed the title Fix numeric compare for Sqlite Fix numeric compare May 6, 2022
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 11 times, most recently from 5c18e4b to b5d0903 Compare May 7, 2022 17:28
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 3 times, most recently from 5910acb to 2b639aa Compare May 30, 2022 12:35
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 8 times, most recently from 699adcf to dbcd00a Compare November 8, 2023 15:47
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 7 times, most recently from 56a4173 to 3a9a0f1 Compare November 11, 2023 22:35
@mvorisek mvorisek removed the MAJOR label Nov 13, 2023
@mvorisek mvorisek force-pushed the sqlite_float_bind branch 3 times, most recently from c666ad2 to 6bd51b7 Compare November 14, 2023 09:43
@mvorisek mvorisek marked this pull request as ready for review November 14, 2023 13:34
@mvorisek mvorisek merged commit e711ab0 into develop Nov 15, 2023
31 checks passed
@mvorisek mvorisek deleted the sqlite_float_bind branch November 15, 2023 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SQLite does not compare integer/float values correctly
1 participant