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

Regression: FB3 throws "Datatypes are not comparable in expression" in procedure parameters [CORE6108] #6357

Closed
firebird-automations opened this issue Jul 24, 2019 · 12 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: Gilberto Ximenes (gilbertoximenes)

Relate to CORE6187

Test case:

SET TERM ^ ;

CREATE OR ALTER procedure TEST_PROC (
P_DATE date)
returns (
OUT_VALUE date)
as
begin
OUT_VALUE = P_DATE;

suspend;
end^

SET TERM ; ^

SELECT OUT_VALUE
FROM TEST_PROC('2019-'|| COALESCE(:ID_MES, 1) ||'-01');
/*
FB2.5 - OK

FB3.0.4.33054-0_x64- Raises
Invalid token.
SQL error code = -104.
Datatypes are not comparable in expression .
*/

Commits: da7a3cc bb0c1e3

====== Test Details ======

Don't close until 3.0.x will be fixed (4.x now is OK).

@firebird-automations
Copy link
Collaborator Author

Modified by: Gilberto Ximenes (gilbertoximenes)

summary: Regression(?) Throws "Datatypes are not comparable in expression" in procedure parameters => Regression: FB3 throws "Datatypes are not comparable in expression" in procedure parameters

security: Developers [ 10012 ] =>

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @asfernandes

Please test the fix in next FB 4 snapshot if possible.

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

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

resolution: Fixed [ 1 ]

Fix Version: 4.0 Beta 2 [ 10888 ]

@firebird-automations
Copy link
Collaborator Author

Commented by: Gilberto Ximenes (gilbertoximenes)

Will it be fixed in FB 3 series too?

@firebird-automations
Copy link
Collaborator Author

Commented by: @asfernandes

Yes, but I would prefer if you validate your real cases first in FB 4 snapshot.

@firebird-automations
Copy link
Collaborator Author

Commented by: Gilberto Ximenes (gilbertoximenes)

Tested in Firebird-4.0.0.1572-0_Win32 and everything works fine. Thank you!

@firebird-automations
Copy link
Collaborator Author

Commented by: @pavel-zotov

Adriano,

i've tested on 4.0.0.1573 x64 - all fine; but 3.0.5.33162 is not yet fixed and bug remains there.

@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 ] => Resolved [ 5 ]

Test Details: Don't close until 3.0.x will be fixed (4.x now is OK).

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

Fix Version: 3.0.5 [ 10885 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Link: This issue relate to CORE6187 [ CORE6187 ]

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

No branches or pull requests

2 participants