Skip to content

Conversation

sebbader-sap
Copy link
Contributor

Solve #477

@mhrimaz
Copy link

mhrimaz commented Jul 9, 2025

I think this requires more consideration.

<selectStatement> ::= "$select" <ws> "id" <ws>

<ws> ::= ( " " | "\t" | "\r" | "\n" )*

Following this grammar both $select id and $selectid will be valid inputs.
Whitespaces should be mandatory in some places and optional in other places.

@sebbader-sap
Copy link
Contributor Author

Following this grammar both $select id and $selectid will be valid inputs.

Yes, that's of course correct. However, as the grammar is intended only as a logical expression of queries, this is acceptable. The syntactic validation of queries must happen on JSON level.

@sebbader-sap sebbader-sap deleted the SeBa/query-grammar-fix branch July 14, 2025 12:29
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 this pull request may close these issues.

3 participants