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.

4 participants