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

Incorrect evaluation of NULL IS [NOT] {FALSE | TRUE} [CORE5986] #6238

Closed
firebird-automations opened this issue Jan 17, 2019 · 8 comments
Closed

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @asfernandes

Is related to CORE5988

- NULL IS {FALSE | TRUE} should be FALSE, but Firebird gives NULL

- NULL IS NOT {FALSE | TRUE} should be TRUE, but Firebird gives NULL

Commits: 4fefe5c 2358514

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Version: 3.0.4 [ 10863 ]

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @mrotteveel

For reference ISO-9075-2:2016 section 6.39 <boolean value expression>, table 15 Truth table for the IS boolean operator.

It might be worthwhile to add an explicit test for those truth tables.

@firebird-automations
Copy link
Collaborator Author

Commented by: @mrotteveel

Note: recent snapshots of Firebird 4 also exhibit the wrong behavior

@firebird-automations
Copy link
Collaborator Author

Modified by: @mrotteveel

Version: 4.0 Beta 1 [ 10750 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Link: This issue is related to CORE5988 [ CORE5988 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Beta 1 [ 10750 ]

Fix Version: 3.0.5 [ 10885 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Done successfully

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment