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

Exp syntax: deprecate case-insensitivity of NULL and booleans #655

Closed
andrus opened this issue Sep 27, 2023 · 0 comments · Fixed by #664
Closed

Exp syntax: deprecate case-insensitivity of NULL and booleans #655

andrus opened this issue Sep 27, 2023 · 0 comments · Fixed by #664
Milestone

Comments

@andrus
Copy link
Contributor

andrus commented Sep 27, 2023

Let's deprecate the use of uppercase literals for NULL, TRUE, FALSE in the expression grammar. They should still work, but a warning should be given in the logs for their use, similar to how we warn about the use of legacy query params:

LOGGER.info("*** 'cayenneExp' control parameter is deprecated in protocol v1.1 (Agrest 4.1). Consider replacing it with 'exp'");
stariy95 added a commit that referenced this issue Jan 31, 2024
…-literals

Deprecate NULL, TRUE, FALSE literals. Fixes #655.
stariy95 added a commit that referenced this issue Jan 31, 2024
#655 Exp syntax: deprecate case-insensitivity of NULL and booleans
@stariy95 stariy95 added this to the 5.0-M20 milestone Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants